* [PATCH] src: consolidate XML/JSON exportation for rule
@ 2017-02-08 15:42 Shivani Bhardwaj
2017-02-09 11:57 ` Arturo Borrero Gonzalez
2017-02-12 13:48 ` Pablo Neira Ayuso
0 siblings, 2 replies; 4+ messages in thread
From: Shivani Bhardwaj @ 2017-02-08 15:42 UTC (permalink / raw)
To: netfilter-devel
This completes the XML/JSON exportation using the new buffer class for
rule.
Signed-off-by: Shivani Bhardwaj <shivanib134@gmail.com>
---
include/buffer.h | 5 +++
src/buffer.c | 11 +++++++
src/rule.c | 96 ++++++++++++++++----------------------------------------
3 files changed, 43 insertions(+), 69 deletions(-)
diff --git a/include/buffer.h b/include/buffer.h
index c571657..a3a50d7 100644
--- a/include/buffer.h
+++ b/include/buffer.h
@@ -35,12 +35,15 @@ int nftnl_buf_u64(struct nftnl_buf *b, int type, uint64_t value, const char *tag
int nftnl_buf_str(struct nftnl_buf *b, int type, const char *str, const char *tag);
int nftnl_buf_reg(struct nftnl_buf *b, int type, union nftnl_data_reg *reg,
int reg_type, const char *tag);
+int nftnl_buf_expr(struct nftnl_buf *b, int type);
#define BASE "base"
#define BYTES "bytes"
#define BURST "burst"
#define CHAIN "chain"
#define CODE "code"
+#define COMPAT_FLAGS "compat_flags"
+#define COMPAT_PROTO "compat_proto"
#define CONSUMED "consumed"
#define DATA "data"
#define DEVICE "device"
@@ -64,10 +67,12 @@ int nftnl_buf_reg(struct nftnl_buf *b, int type, union nftnl_data_reg *reg,
#define PACKETS "packets"
#define PKTS "pkts"
#define POLICY "policy"
+#define POSITION "position"
#define PREFIX "prefix"
#define PRIO "prio"
#define QTHRESH "qthreshold"
#define RATE "rate"
+#define RULE "rule"
#define SET "set"
#define SET_NAME "set_name"
#define SIZE "size"
diff --git a/src/buffer.c b/src/buffer.c
index d97d517..8dffef2 100644
--- a/src/buffer.c
+++ b/src/buffer.c
@@ -171,3 +171,14 @@ int nftnl_buf_reg(struct nftnl_buf *b, int type, union nftnl_data_reg *reg,
}
return 0;
}
+
+int nftnl_buf_expr(struct nftnl_buf *b, int type)
+{
+ switch (type) {
+ case NFTNL_OUTPUT_XML:
+ return 0;
+ case NFTNL_OUTPUT_JSON:
+ return nftnl_buf_put(b, "\"expr\":[{");
+ }
+ return 0;
+}
diff --git a/src/rule.c b/src/rule.c
index 02d013b..5359972 100644
--- a/src/rule.c
+++ b/src/rule.c
@@ -20,6 +20,7 @@
#include <errno.h>
#include <inttypes.h>
#include <ctype.h>
+#include <buffer.h>
#include <libmnl/libmnl.h>
#include <linux/netfilter/nfnetlink.h>
@@ -645,83 +646,40 @@ int nftnl_rule_parse_file(struct nftnl_rule *r, enum nftnl_parse_type type,
}
EXPORT_SYMBOL(nftnl_rule_parse_file);
-static int nftnl_rule_snprintf_json(char *buf, size_t size,
- const struct nftnl_rule *r,
- uint32_t type, uint32_t flags)
+static int nftnl_rule_export(char *buf, size_t size,
+ const struct nftnl_rule *r,
+ uint32_t type, uint32_t flags)
{
- int ret, len = size, offset = 0;
struct nftnl_expr *expr;
- ret = snprintf(buf, len, "{\"rule\":{");
- SNPRINTF_BUFFER_SIZE(ret, size, len, offset);
-
- if (r->flags & (1 << NFTNL_RULE_FAMILY)) {
- ret = snprintf(buf+offset, len, "\"family\":\"%s\",",
- nftnl_family2str(r->family));
- SNPRINTF_BUFFER_SIZE(ret, size, len, offset);
- }
-
- if (r->flags & (1 << NFTNL_RULE_TABLE)) {
- ret = snprintf(buf+offset, len, "\"table\":\"%s\",",
- r->table);
- SNPRINTF_BUFFER_SIZE(ret, size, len, offset);
- }
-
- if (r->flags & (1 << NFTNL_RULE_CHAIN)) {
- ret = snprintf(buf+offset, len, "\"chain\":\"%s\",",
- r->chain);
- SNPRINTF_BUFFER_SIZE(ret, size, len, offset);
- }
- if (r->flags & (1 << NFTNL_RULE_HANDLE)) {
- ret = snprintf(buf+offset, len, "\"handle\":%llu,",
- (unsigned long long)r->handle);
- SNPRINTF_BUFFER_SIZE(ret, size, len, offset);
- }
-
- if (r->flags & (1 << NFTNL_RULE_COMPAT_PROTO) ||
- r->flags & (1 << NFTNL_RULE_COMPAT_FLAGS)) {
- ret = snprintf(buf+offset, len, "\"compat_flags\":%u,"
- "\"compat_proto\":%u,",
- r->compat.flags, r->compat.proto);
- SNPRINTF_BUFFER_SIZE(ret, size, len, offset);
- }
+ NFTNL_BUF_INIT(b, buf, size);
- if (r->flags & (1 << NFTNL_RULE_POSITION)) {
- ret = snprintf(buf+offset, len, "\"position\":%"PRIu64",",
- r->position);
- SNPRINTF_BUFFER_SIZE(ret, size, len, offset);
- }
+ nftnl_buf_open(&b, type, RULE);
+ if (r->flags & (1 << NFTNL_RULE_FAMILY))
+ nftnl_buf_str(&b, type, nftnl_family2str(r->family), FAMILY);
+ if (r->flags & (1 << NFTNL_RULE_TABLE))
+ nftnl_buf_str(&b, type, r->table, TABLE);
+ if (r->flags & (1 << NFTNL_RULE_CHAIN))
+ nftnl_buf_str(&b, type, r->chain, CHAIN);
+ if (r->flags & (1 << NFTNL_RULE_HANDLE))
+ nftnl_buf_u64(&b, type, r->handle, HANDLE);
+ if (r->flags & (1 << NFTNL_RULE_COMPAT_PROTO))
+ nftnl_buf_u32(&b, type, r->compat.proto, COMPAT_PROTO);
+ if (r->flags & (1 << NFTNL_RULE_COMPAT_FLAGS))
+ nftnl_buf_u32(&b, type, r->compat.flags, COMPAT_FLAGS);
+ if (r->flags & (1 << NFTNL_RULE_POSITION))
+ nftnl_buf_u64(&b, type, r->position, POSITION);
- ret = snprintf(buf+offset, len, "\"expr\":[");
- SNPRINTF_BUFFER_SIZE(ret, size, len, offset);
+ nftnl_buf_expr(&b, type);
list_for_each_entry(expr, &r->expr_list, head) {
- ret = snprintf(buf+offset, len,
- "{\"type\":\"%s\",", expr->ops->name);
- SNPRINTF_BUFFER_SIZE(ret, size, len, offset);
-
- ret = expr->ops->snprintf(buf+offset, len, type, flags, expr);
- SNPRINTF_BUFFER_SIZE(ret, size, len, offset);
-
- /*
- * Remove comma from the first element if there is type
- * key-value pair only. Example: "expr":[{"type":"log"}]
- */
- if (ret == 0) {
- offset--;
- len--;
- }
-
- ret = snprintf(buf+offset, len, "},");
- SNPRINTF_BUFFER_SIZE(ret, size, len, offset);
-
+ nftnl_buf_str(&b, type, expr->ops->name, TYPE);
+ vsnprintf(b.buf, b.len, flags, expr);
}
- /* Remove comma from last element */
- offset--;
- ret = snprintf(buf+offset, len, "]}}");
- SNPRINTF_BUFFER_SIZE(ret, size, len, offset);
- return offset;
+ nftnl_buf_close(&b, type, RULE);
+
+ return nftnl_buf_done(&b);
}
static int nftnl_rule_snprintf_default(char *buf, size_t size,
@@ -813,7 +771,7 @@ static int nftnl_rule_cmd_snprintf(char *buf, size_t size,
inner_flags);
break;
case NFTNL_OUTPUT_JSON:
- ret = nftnl_rule_snprintf_json(buf+offset, len, r, type,
+ ret = nftnl_rule_export(buf+offset, len, r, type,
inner_flags);
break;
case NFTNL_OUTPUT_XML:
--
2.7.4
^ permalink raw reply related [flat|nested] 4+ messages in thread
* Re: [PATCH] src: consolidate XML/JSON exportation for rule
2017-02-08 15:42 [PATCH] src: consolidate XML/JSON exportation for rule Shivani Bhardwaj
@ 2017-02-09 11:57 ` Arturo Borrero Gonzalez
2017-02-12 13:48 ` Pablo Neira Ayuso
1 sibling, 0 replies; 4+ messages in thread
From: Arturo Borrero Gonzalez @ 2017-02-09 11:57 UTC (permalink / raw)
To: Netfilter Development Mailing list; +Cc: Shivani Bhardwaj
On 8 February 2017 at 16:42, Shivani Bhardwaj <shivanib134@gmail.com> wrote:
> This completes the XML/JSON exportation using the new buffer class for
> rule.
>
> Signed-off-by: Shivani Bhardwaj <shivanib134@gmail.com>
> ---
> include/buffer.h | 5 +++
> src/buffer.c | 11 +++++++
> src/rule.c | 96 ++++++++++++++++----------------------------------------
> 3 files changed, 43 insertions(+), 69 deletions(-)
>
Acked-by: Arturo Borrero Gonzalez <arturo@debian.org>
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH] src: consolidate XML/JSON exportation for rule
2017-02-08 15:42 [PATCH] src: consolidate XML/JSON exportation for rule Shivani Bhardwaj
2017-02-09 11:57 ` Arturo Borrero Gonzalez
@ 2017-02-12 13:48 ` Pablo Neira Ayuso
2017-02-12 13:50 ` Pablo Neira Ayuso
1 sibling, 1 reply; 4+ messages in thread
From: Pablo Neira Ayuso @ 2017-02-12 13:48 UTC (permalink / raw)
To: Shivani Bhardwaj; +Cc: netfilter-devel
On Wed, Feb 08, 2017 at 09:12:53PM +0530, Shivani Bhardwaj wrote:
> This completes the XML/JSON exportation using the new buffer class for
> rule.
Applied, thanks Shivani.
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH] src: consolidate XML/JSON exportation for rule
2017-02-12 13:48 ` Pablo Neira Ayuso
@ 2017-02-12 13:50 ` Pablo Neira Ayuso
0 siblings, 0 replies; 4+ messages in thread
From: Pablo Neira Ayuso @ 2017-02-12 13:50 UTC (permalink / raw)
To: Shivani Bhardwaj; +Cc: netfilter-devel
On Sun, Feb 12, 2017 at 02:48:43PM +0100, Pablo Neira Ayuso wrote:
> On Wed, Feb 08, 2017 at 09:12:53PM +0530, Shivani Bhardwaj wrote:
> > This completes the XML/JSON exportation using the new buffer class for
> > rule.
>
> Applied, thanks Shivani.
Hm, there's some problem here, look:
drule.c: In function 'nftnl_rule_export':
rule.c:677:27: warning: passing argument 3 of 'vsnprintf' makes pointer from integer without a cast
vsnprintf(b.buf, b.len, flags, expr);
^
In file included from ../include/utils.h:4:0,
from ../include/internal.h:7,
from rule.c:11:
/usr/include/stdio.h:390:12: note: expected 'const char *
__restrict__' but argument is of type 'uint32_t' extern int vsnprintf (char *__restrict __s, size_t __maxlen,
^
rule.c:677:34: warning: passing argument 4 of 'vsnprintf' from incompatible pointer type vsnprintf(b.buf, b.len, flags, expr);
vsnprintf(b.buf, b.len, flags, expr);
^
In file included from ../include/utils.h:4:0,
from ../include/internal.h:7,
from rule.c:11:
/usr/include/stdio.h:390:12: note: expected 'struct __va_list_tag *' but argument is of type 'struct nftnl_expr *'
extern int vsnprintf (char *__restrict __s, size_t __maxlen,
^
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2017-02-12 13:50 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2017-02-08 15:42 [PATCH] src: consolidate XML/JSON exportation for rule Shivani Bhardwaj
2017-02-09 11:57 ` Arturo Borrero Gonzalez
2017-02-12 13:48 ` Pablo Neira Ayuso
2017-02-12 13:50 ` Pablo Neira Ayuso
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.