From acc963410636a417b975eb954f7369b3c4f7f5d1 Mon Sep 17 00:00:00 2001
From: Eric Leblond <eric@regit.org>
Date: Tue, 29 Nov 2011 10:17:02 +0100
Subject: [PATCH] nfq: add some comments about possible evolution

---
 src/source-nfq.c | 7 +++++++
 1 file changed, 7 insertions(+)

diff --git a/src/source-nfq.c b/src/source-nfq.c
index 5e1515442d..d576b44b32 100644
--- a/src/source-nfq.c
+++ b/src/source-nfq.c
@@ -636,6 +636,8 @@ void *NFQGetQueue(int number) {
 /**
  *  \brief Get a pointer to the NFQ thread at index
  *
+ *  This function is temporary used as configuration parser.
+ *
  *  \param number idx of the queue in our array
  *
  *  \retval ptr pointer to the NFQThreadVars at index
@@ -832,8 +834,13 @@ TmEcode NFQSetVerdict(Packet *p) {
     int iter = 0;
     int ret = 0;
     uint32_t verdict = NF_ACCEPT;
+    /* we could also have a direct pointer but we need to have a ref counf in this case */
     NFQQueueVars *t = nfq_q + p->nfq_v.nfq_index;
 
+    /** \todo add a test on validity of the entry NFQQueueVars could have been
+     *  wipeout
+     */
+
     /* can't verdict a "fake" packet */
     if (p->flags & PKT_PSEUDO_STREAM_END) {
         return TM_ECODE_OK;