diff options
author | Jeremy Huddleston Sequoia <jeremyhu@apple.com> | 2014-01-02 00:49:27 -0800 |
---|---|---|
committer | Jeremy Huddleston Sequoia <jeremyhu@apple.com> | 2014-01-02 00:49:27 -0800 |
commit | 4b0b74f953619164dc2863ee2cd3f09d15c65a51 (patch) | |
tree | 3f28636576977ac62588014e41406c836eaa81e4 | |
parent | 5ed8b4cf1f1c35ef35a3c94350e6b96646a2a0b7 (diff) |
Update error loop to correctly handle error in i==0 case
FSOpenServ.c:183:3: warning: Function call argument is an uninitialized value
FSfree(alts[i].name);
^~~~~~~~~~~~~~~~~~~~
./FSlibos.h:273:21: note: expanded from macro 'FSfree'
^~~~~~~~~~~
Signed-off-by: Jeremy Huddleston Sequoia <jeremyhu@apple.com>
-rw-r--r-- | src/FSOpenServ.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/FSOpenServ.c b/src/FSOpenServ.c index 15a657a..9b51e20 100644 --- a/src/FSOpenServ.c +++ b/src/FSOpenServ.c @@ -179,7 +179,7 @@ FSOpenServer(const char *server) altlen = (unsigned int) *ad++; alts[i].name = FSmalloc(altlen + 1); if (!alts[i].name) { - while (--i) { + while (--i >= 0) { FSfree(alts[i].name); } goto fail; |