From 7349a70634fa02838aea33be0ff1b5491ac8dee9 Mon Sep 17 00:00:00 2001 From: Phil Sutter Date: Wed, 30 Oct 2019 18:31:59 +0100 Subject: Deprecate untyped data setters These functions make assumptions on size of passed data pointer and therefore tend to hide programming mistakes. Instead either one of the type-specific setters or the generic *_set_data() setter should be used. Signed-off-by: Phil Sutter Acked-by: Pablo Neira Ayuso --- include/libnftnl/chain.h | 2 +- include/libnftnl/flowtable.h | 2 +- include/libnftnl/gen.h | 2 +- include/libnftnl/object.h | 2 +- include/libnftnl/rule.h | 2 +- include/libnftnl/set.h | 2 +- include/libnftnl/table.h | 2 +- 7 files changed, 7 insertions(+), 7 deletions(-) (limited to 'include/libnftnl') diff --git a/include/libnftnl/chain.h b/include/libnftnl/chain.h index f84a2a3..33d04e1 100644 --- a/include/libnftnl/chain.h +++ b/include/libnftnl/chain.h @@ -38,7 +38,7 @@ enum nftnl_chain_attr { bool nftnl_chain_is_set(const struct nftnl_chain *c, uint16_t attr); void nftnl_chain_unset(struct nftnl_chain *c, uint16_t attr); -void nftnl_chain_set(struct nftnl_chain *t, uint16_t attr, const void *data); +void nftnl_chain_set(struct nftnl_chain *t, uint16_t attr, const void *data) __attribute__((deprecated)); int nftnl_chain_set_data(struct nftnl_chain *t, uint16_t attr, const void *data, uint32_t data_len); void nftnl_chain_set_u8(struct nftnl_chain *t, uint16_t attr, uint8_t data); diff --git a/include/libnftnl/flowtable.h b/include/libnftnl/flowtable.h index 6f68018..028095e 100644 --- a/include/libnftnl/flowtable.h +++ b/include/libnftnl/flowtable.h @@ -33,7 +33,7 @@ enum nftnl_flowtable_attr { bool nftnl_flowtable_is_set(const struct nftnl_flowtable *c, uint16_t attr); void nftnl_flowtable_unset(struct nftnl_flowtable *c, uint16_t attr); -void nftnl_flowtable_set(struct nftnl_flowtable *t, uint16_t attr, const void *data); +void nftnl_flowtable_set(struct nftnl_flowtable *t, uint16_t attr, const void *data) __attribute__((deprecated)); int nftnl_flowtable_set_data(struct nftnl_flowtable *t, uint16_t attr, const void *data, uint32_t data_len); void nftnl_flowtable_set_u32(struct nftnl_flowtable *t, uint16_t attr, uint32_t data); diff --git a/include/libnftnl/gen.h b/include/libnftnl/gen.h index c56a63c..846b8e0 100644 --- a/include/libnftnl/gen.h +++ b/include/libnftnl/gen.h @@ -25,7 +25,7 @@ enum { bool nftnl_gen_is_set(const struct nftnl_gen *gen, uint16_t attr); void nftnl_gen_unset(struct nftnl_gen *gen, uint16_t attr); -int nftnl_gen_set(struct nftnl_gen *gen, uint16_t attr, const void *data); +int nftnl_gen_set(struct nftnl_gen *gen, uint16_t attr, const void *data) __attribute__((deprecated)); int nftnl_gen_set_data(struct nftnl_gen *gen, uint16_t attr, const void *data, uint32_t data_len); const void *nftnl_gen_get(const struct nftnl_gen *gen, uint16_t attr); diff --git a/include/libnftnl/object.h b/include/libnftnl/object.h index c5ea88e..221b15c 100644 --- a/include/libnftnl/object.h +++ b/include/libnftnl/object.h @@ -124,7 +124,7 @@ bool nftnl_obj_is_set(const struct nftnl_obj *ne, uint16_t attr); void nftnl_obj_unset(struct nftnl_obj *ne, uint16_t attr); void nftnl_obj_set_data(struct nftnl_obj *ne, uint16_t attr, const void *data, uint32_t data_len); -void nftnl_obj_set(struct nftnl_obj *ne, uint16_t attr, const void *data); +void nftnl_obj_set(struct nftnl_obj *ne, uint16_t attr, const void *data) __attribute__((deprecated)); void nftnl_obj_set_u8(struct nftnl_obj *ne, uint16_t attr, uint8_t val); void nftnl_obj_set_u16(struct nftnl_obj *ne, uint16_t attr, uint16_t val); void nftnl_obj_set_u32(struct nftnl_obj *ne, uint16_t attr, uint32_t val); diff --git a/include/libnftnl/rule.h b/include/libnftnl/rule.h index 78bfead..e5d1ca0 100644 --- a/include/libnftnl/rule.h +++ b/include/libnftnl/rule.h @@ -35,7 +35,7 @@ enum nftnl_rule_attr { void nftnl_rule_unset(struct nftnl_rule *r, uint16_t attr); bool nftnl_rule_is_set(const struct nftnl_rule *r, uint16_t attr); -int nftnl_rule_set(struct nftnl_rule *r, uint16_t attr, const void *data); +int nftnl_rule_set(struct nftnl_rule *r, uint16_t attr, const void *data) __attribute__((deprecated)); int nftnl_rule_set_data(struct nftnl_rule *r, uint16_t attr, const void *data, uint32_t data_len); void nftnl_rule_set_u32(struct nftnl_rule *r, uint16_t attr, uint32_t val); diff --git a/include/libnftnl/set.h b/include/libnftnl/set.h index 2ea2e9a..db3fa68 100644 --- a/include/libnftnl/set.h +++ b/include/libnftnl/set.h @@ -42,7 +42,7 @@ struct nftnl_set *nftnl_set_clone(const struct nftnl_set *set); bool nftnl_set_is_set(const struct nftnl_set *s, uint16_t attr); void nftnl_set_unset(struct nftnl_set *s, uint16_t attr); -int nftnl_set_set(struct nftnl_set *s, uint16_t attr, const void *data); +int nftnl_set_set(struct nftnl_set *s, uint16_t attr, const void *data) __attribute__((deprecated)); int nftnl_set_set_data(struct nftnl_set *s, uint16_t attr, const void *data, uint32_t data_len); void nftnl_set_set_u32(struct nftnl_set *s, uint16_t attr, uint32_t val); diff --git a/include/libnftnl/table.h b/include/libnftnl/table.h index 44017c6..5faec81 100644 --- a/include/libnftnl/table.h +++ b/include/libnftnl/table.h @@ -29,7 +29,7 @@ enum nftnl_table_attr { bool nftnl_table_is_set(const struct nftnl_table *t, uint16_t attr); void nftnl_table_unset(struct nftnl_table *t, uint16_t attr); -void nftnl_table_set(struct nftnl_table *t, uint16_t attr, const void *data); +void nftnl_table_set(struct nftnl_table *t, uint16_t attr, const void *data) __attribute__((deprecated)); int nftnl_table_set_data(struct nftnl_table *t, uint16_t attr, const void *data, uint32_t data_len); const void *nftnl_table_get(const struct nftnl_table *t, uint16_t attr); -- cgit v1.2.3