Stefan Reinauer has uploaded this change for review. ( https://review.coreboot.org/c/em100/+/37422 )
Change subject: Fix CID 264288 Resource leak ......................................................................
Fix CID 264288 Resource leak
opendir() needs closedir().
Signed-off-by: Stefan Reinauer stefan.reinauer@coreboot.org Change-Id: I316ee4f83a8b964cd9b0a6a5c3142ee080cab747 --- M em100.c 1 file changed, 1 insertion(+), 0 deletions(-)
git pull ssh://review.coreboot.org:29418/em100 refs/changes/22/37422/1
diff --git a/em100.c b/em100.c index 301d574..5fa95dd 100644 --- a/em100.c +++ b/em100.c @@ -733,6 +733,7 @@ return NULL; }
+ closedir(dir); return directory; }
Patrick Georgi has posted comments on this change. ( https://review.coreboot.org/c/em100/+/37422 )
Change subject: Fix CID 264288 Resource leak ......................................................................
Patch Set 1: Code-Review+2
Stefan Reinauer has submitted this change. ( https://review.coreboot.org/c/em100/+/37422 )
Change subject: Fix CID 264288 Resource leak ......................................................................
Fix CID 264288 Resource leak
opendir() needs closedir().
Signed-off-by: Stefan Reinauer stefan.reinauer@coreboot.org Change-Id: I316ee4f83a8b964cd9b0a6a5c3142ee080cab747 Reviewed-on: https://review.coreboot.org/c/em100/+/37422 Tested-by: build bot (Jenkins) no-reply@coreboot.org Reviewed-by: Patrick Georgi pgeorgi@google.com --- M em100.c 1 file changed, 1 insertion(+), 0 deletions(-)
Approvals: build bot (Jenkins): Verified Patrick Georgi: Looks good to me, approved
diff --git a/em100.c b/em100.c index 301d574..5fa95dd 100644 --- a/em100.c +++ b/em100.c @@ -733,6 +733,7 @@ return NULL; }
+ closedir(dir); return directory; }
-
Patrick Georgi (Code Review) -
Stefan Reinauer (Code Review)