On 01/07/16 16:22, Pablo Neira Ayuso wrote:
On Mon, Jun 27, 2016 at 06:24:25PM +0200, Carlos Falgueras García wrote:
We need the string length

Signed-off-by: Carlos Falgueras García <carlo...@riseup.net>
---
  src/set.c | 2 +-
  1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/set.c b/src/set.c
index 879100c..edbcbe5 100644
--- a/src/set.c
+++ b/src/set.c
@@ -203,7 +203,7 @@ EXPORT_SYMBOL_ALIAS(nftnl_set_set_u64, 
nft_set_attr_set_u64);
int nftnl_set_set_str(struct nftnl_set *s, uint16_t attr, const char *str)
  {
-       return nftnl_set_set(s, attr, str);
+       return nftnl_set_set_data(s, attr, str, strlen(str));
I think this should be strlen(str) + 1 so we make sure the string is
nul-terminated.
Thanks Pablo. I have sent a v2 and another patch to fix this error in other code parts.
--
To unsubscribe from this list: send the line "unsubscribe netfilter-devel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to