On 08/08/2016 01:25 PM, Pablo Neira Ayuso wrote:
On Mon, Aug 08, 2016 at 01:17:55PM +0200, Carlos Falgueras García wrote:
With 'nftnl_expr_iter_init' we can create an expression iterator without
dynamic memory allocation.
I'd suggest this description:
Introduce nftnl_expr_iter_init() to allow stack allocated iterators
for internal use.
Another comment below.
Ok, I'll change it at v2, thanks.
Signed-off-by: Carlos Falgueras García <carlosfg@xxxxxxxxxx>
---
include/internal.h | 1 +
include/rule.h | 15 +++++++++++++++
src/rule.c | 23 ++++++++++++-----------
3 files changed, 28 insertions(+), 11 deletions(-)
create mode 100644 include/rule.h
diff --git a/include/internal.h b/include/internal.h
index c74e2bf..f1b6511 100644
--- a/include/internal.h
+++ b/include/internal.h
@@ -12,6 +12,7 @@
#include "set.h"
#include "set_elem.h"
#include "expr.h"
+#include "rule.h"
#include "expr_ops.h"
#include "buffer.h"
diff --git a/include/rule.h b/include/rule.h
new file mode 100644
index 0000000..e2ea578
--- /dev/null
+++ b/include/rule.h
@@ -0,0 +1,15 @@
+#ifndef _LIBNFTNL_RULE_INTERNAL_H_
+#define _LIBNFTNL_RULE_INTERNAL_H_
+
+#include <libnftnl/rule.h>
+#include <libnftnl/expr.h>
+
+struct nftnl_expr_iter {
+ const struct nftnl_rule *r;
+ struct nftnl_expr *cur;
+};
+
+void nftnl_expr_iter_init(const struct nftnl_rule *r,
+ struct nftnl_expr_iter *iter);
If nftnl_expr_iter_init() is only used from src/rule.c, then there is
no need to expose this rule include/rule.h
Ok.
--
To unsubscribe from this list: send the line "unsubscribe netfilter-devel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html