diff options
| author | Jason Evans <je@fb.com> | 2013-10-21 22:01:44 (GMT) |
|---|---|---|
| committer | Jason Evans <je@fb.com> | 2013-10-21 22:01:44 (GMT) |
| commit | e2985a23819670866c041ba07964099eeb9e0e07 (patch) | |
| tree | 7aab1cf673e8cf977e105d85aeb99d6d52241e26 /src/jemalloc.c | |
| parent | 30e7cb11186554eb3ee860856eb5b8d541d7740c (diff) | |
| download | jemalloc-e2985a23819670866c041ba07964099eeb9e0e07.zip jemalloc-e2985a23819670866c041ba07964099eeb9e0e07.tar.gz jemalloc-e2985a23819670866c041ba07964099eeb9e0e07.tar.bz2 | |
Avoid (x < 0) comparison for unsigned x.
Avoid (min < 0) comparison for unsigned min in malloc_conf_init(). This
bug had no practical consequences.
Reported by Pat Lynch.
Diffstat (limited to 'src/jemalloc.c')
| -rw-r--r-- | src/jemalloc.c | 5 |
1 files changed, 3 insertions, 2 deletions
diff --git a/src/jemalloc.c b/src/jemalloc.c index 5cbfc3f..eb4bb7b 100644 --- a/src/jemalloc.c +++ b/src/jemalloc.c @@ -521,14 +521,15 @@ malloc_conf_init(void) "Invalid conf value", \ k, klen, v, vlen); \ } else if (clip) { \ - if (um < min) \ + if (min != 0 && um < min) \ o = min; \ else if (um > max) \ o = max; \ else \ o = um; \ } else { \ - if (um < min || um > max) { \ + if ((min != 0 && um < min) || \ + um > max) { \ malloc_conf_error( \ "Out-of-range " \ "conf value", \ |