From patchwork Tue Dec 25 14:21:10 2012 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit Subject: [libnl-nft] Fix file descriptor leak on error Date: Tue, 25 Dec 2012 04:21:10 -0000 From: Thomas Jarosch X-Patchwork-Id: 208107 Message-Id: <50D9B656.6040809@intra2net.com> To: Patrick McHardy Cc: netfilter-devel@vger.kernel.org Detected by cppcheck Signed-off-by: Thomas Jarosch --- lib/utils.c | 16 ++++++++++++---- 1 file changed, 12 insertions(+), 4 deletions(-) diff --git a/lib/utils.c b/lib/utils.c index 4007bee..50ca6d9 100644 --- a/lib/utils.c +++ b/lib/utils.c @@ -60,24 +60,32 @@ int __nl_read_num_str_file(const char *path, int (*cb)(long, const char *)) continue; num = strtol(buf, &end, 0); - if (end == buf) + if (end == buf) { + fclose(fd); return -NLE_INVAL; + } - if (num == LONG_MIN || num == LONG_MAX) + if (num == LONG_MIN || num == LONG_MAX) { + fclose(fd); return -NLE_RANGE; + } while (*end == ' ' || *end == '\t') end++; goodlen = strcspn(end, "#\r\n\t "); - if (goodlen == 0) + if (goodlen == 0) { + fclose(fd); return -NLE_INVAL; + } end[goodlen] = '\0'; err = cb(num, end); - if (err < 0) + if (err < 0) { + fclose(fd); return err; + } } fclose(fd);