lib:util: Fix size types in fgets_slash()
authorAndreas Schneider <asn@samba.org>
Wed, 21 Mar 2018 11:49:38 +0000 (12:49 +0100)
committerJeremy Allison <jra@samba.org>
Tue, 3 Apr 2018 18:20:10 +0000 (20:20 +0200)
This fixes compilation with -Wstrict-overflow=2.

Signed-off-by: Andreas Schneider <asn@samba.org>
Reviewed-by: Jeremy Allison <jra@samba.org>
lib/util/samba_util.h
lib/util/util_file.c

index b78252316c80457462a124c13fa45e67772913d1..7b96a595d437a3d9494a1c6733d38052dc4d113b 100644 (file)
@@ -352,7 +352,7 @@ const char **str_list_make_v3_const(TALLOC_CTX *mem_ctx,
  */
 _PUBLIC_ char *afdgets(int fd, TALLOC_CTX *mem_ctx, size_t hint);
 
-char *fgets_slash(TALLOC_CTX *mem_ctx, char *s2, int maxlen, FILE *f);
+char *fgets_slash(TALLOC_CTX *mem_ctx, char *s2, size_t maxlen, FILE *f);
 
 /**
 load a file into memory from a fd.
index 499e8c466939bec9a56af19997f4986ccc45928d..bf2f3e1a27f42135f1251a28975dbdc45cba5597 100644 (file)
@@ -79,10 +79,10 @@ _PUBLIC_ char *afdgets(int fd, TALLOC_CTX *mem_ctx, size_t hint)
        return data;
 }
 
-char *fgets_slash(TALLOC_CTX *mem_ctx, char *s2, int maxlen, FILE *f)
+char *fgets_slash(TALLOC_CTX *mem_ctx, char *s2, size_t maxlen, FILE *f)
 {
        char *s = s2;
-       int len = 0;
+       size_t len = 0;
        int c;
        bool start_of_line = true;