Static checker revealed possible resource/memory leaks
of file handle and process pipe.
Signed-off-by: Thomas Stibor <t.stibor@gsi.de>
Change-Id: I4ee05a8325408370bcfb5e9b8d30253107115c7b
Reviewed-on: https://review.whamcloud.com/34129
Tested-by: Jenkins
Reviewed-by: Sebastien Buisson <sbuisson@ddn.com>
Reviewed-by: Shilong Wang <wshilong@ddn.com>
Tested-by: Maloo <maloo@whamcloud.com>
Reviewed-by: Oleg Drokin <green@whamcloud.com>
head = fgets(buf, bufsize, fghn);
if (head == NULL) {
printerr(0, "can't read from %s\n", gethostname_ex);
+ pclose(fghn);
return -1;
}
if (pclose(fghn) == -1)
if (ret) {
fprintf(stderr, "%s: failed to get xattr for '%s': %s\n",
progname, pathname, strerror(errno));
+ close(fd);
return -errno;
}
return fd;