From 158a5b97a312f6810912d0e31ded4275a6a787ff Mon Sep 17 00:00:00 2001 From: Pablo Neira Ayuso Date: Mon, 19 Apr 2010 17:59:31 +0200 Subject: review data types for input parameters of mnl_attr_*() functions This patch includes the following changes: * use uint16_t for attribute types in mnl_attr_put_*() function * use uint16_t for maximun type in mnl_attr_type_valid(...) * use size_t for attribute size validation in mnl_attr_validate2(...) * use unsigned int for offset in mnl_attr_parse(...) instead of int Signed-off-by: Pablo Neira Ayuso --- include/libmnl/libmnl.h | 20 ++++++++++---------- 1 file changed, 10 insertions(+), 10 deletions(-) (limited to 'include/libmnl/libmnl.h') diff --git a/include/libmnl/libmnl.h b/include/libmnl/libmnl.h index cec6304..af7d30b 100644 --- a/include/libmnl/libmnl.h +++ b/include/libmnl/libmnl.h @@ -77,16 +77,16 @@ extern uint64_t mnl_attr_get_u64(const struct nlattr *attr); extern const char *mnl_attr_get_str(const struct nlattr *attr); /* TLV attribute putters */ -extern void mnl_attr_put(struct nlmsghdr *nlh, int type, size_t len, const void *data); -extern void mnl_attr_put_u8(struct nlmsghdr *nlh, int type, uint8_t data); -extern void mnl_attr_put_u16(struct nlmsghdr *nlh, int type, uint16_t data); -extern void mnl_attr_put_u32(struct nlmsghdr *nlh, int type, uint32_t data); -extern void mnl_attr_put_u64(struct nlmsghdr *nlh, int type, uint64_t data); -extern void mnl_attr_put_str(struct nlmsghdr *nlh, int type, const void *data); -extern void mnl_attr_put_str_null(struct nlmsghdr *nlh, int type, const void *data); +extern void mnl_attr_put(struct nlmsghdr *nlh, uint16_t type, size_t len, const void *data); +extern void mnl_attr_put_u8(struct nlmsghdr *nlh, uint16_t type, uint8_t data); +extern void mnl_attr_put_u16(struct nlmsghdr *nlh, uint16_t type, uint16_t data); +extern void mnl_attr_put_u32(struct nlmsghdr *nlh, uint16_t type, uint32_t data); +extern void mnl_attr_put_u64(struct nlmsghdr *nlh, uint16_t type, uint64_t data); +extern void mnl_attr_put_str(struct nlmsghdr *nlh, uint16_t type, const void *data); +extern void mnl_attr_put_str_null(struct nlmsghdr *nlh, uint16_t type, const void *data); /* TLV validation */ -extern int mnl_attr_type_valid(const struct nlattr *attr, int maxtype); +extern int mnl_attr_type_valid(const struct nlattr *attr, uint16_t maxtype); enum mnl_attr_data_type { MNL_TYPE_UNSPEC, @@ -105,7 +105,7 @@ enum mnl_attr_data_type { }; extern int mnl_attr_validate(const struct nlattr *attr, enum mnl_attr_data_type type); -extern int mnl_attr_validate2(const struct nlattr *attr, enum mnl_attr_data_type type, int minlen); +extern int mnl_attr_validate2(const struct nlattr *attr, enum mnl_attr_data_type type, size_t len); /* TLV iterators */ extern int mnl_attr_ok(const struct nlattr *attr, int len); @@ -126,7 +126,7 @@ extern struct nlattr *mnl_attr_next(const struct nlattr *attr, int *len); /* TLV callback-based attribute parsers */ typedef int (*mnl_attr_cb_t)(const struct nlattr *attr, void *data); -extern int mnl_attr_parse(const struct nlmsghdr *nlh, int offset, mnl_attr_cb_t cb, void *data); +extern int mnl_attr_parse(const struct nlmsghdr *nlh, unsigned int offset, mnl_attr_cb_t cb, void *data); extern int mnl_attr_parse_nested(const struct nlattr *attr, mnl_attr_cb_t cb, void *data); /* -- cgit v1.2.3