diff options
| author | uqs <uqs@FreeBSD.org> | 2015-12-29 11:24:41 +0000 |
|---|---|---|
| committer | uqs <uqs@FreeBSD.org> | 2015-12-29 11:24:41 +0000 |
| commit | 11e0dd6c90098a31b4ee84a2af39114001952d67 (patch) | |
| tree | b886de70650826ff015bc0343d808ffba0f524fd /usr.bin/locate | |
| parent | d4f2c120c09150a7012ceea91a74ea068aa5a9c6 (diff) | |
| download | FreeBSD-src-11e0dd6c90098a31b4ee84a2af39114001952d67.zip FreeBSD-src-11e0dd6c90098a31b4ee84a2af39114001952d67.tar.gz | |
Fix type mismatches for malloc(3) and Co.
This is rather pedantic, as for most architectures it holds that
sizeof(type *) == sizeof(type **)
Found by: clang static analyzer
Reviewed by: ed
Differential Revision: https://reviews.freebsd.org/D4722
Diffstat (limited to 'usr.bin/locate')
| -rw-r--r-- | usr.bin/locate/locate/util.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/usr.bin/locate/locate/util.c b/usr.bin/locate/locate/util.c index 3ac69b5..9130e37 100644 --- a/usr.bin/locate/locate/util.c +++ b/usr.bin/locate/locate/util.c @@ -93,7 +93,7 @@ colon(dbv, path, dot) char **pv; if (dbv == NULL) { - if ((dbv = malloc(sizeof(char **))) == NULL) + if ((dbv = malloc(sizeof(char *))) == NULL) err(1, "malloc"); *dbv = NULL; } @@ -123,7 +123,7 @@ colon(dbv, path, dot) *(p + slen) = '\0'; } /* increase dbv with element p */ - if ((dbv = realloc(dbv, sizeof(char **) * (vlen + 2))) + if ((dbv = realloc(dbv, sizeof(char *) * (vlen + 2))) == NULL) err(1, "realloc"); *(dbv + vlen) = p; |
