p2pd: do not close an fd that is not open
authorFerry Huberts <ferry.huberts@pelagic.nl>
Tue, 23 Oct 2012 10:10:30 +0000 (12:10 +0200)
committerFerry Huberts <ferry.huberts@pelagic.nl>
Tue, 23 Oct 2012 11:45:14 +0000 (13:45 +0200)
Coverity:
CID 739656 (#5 of 8): Argument cannot be negative (NEGATIVE_RETURNS)
At (6): "encapsulatingSkfd" is passed to a parameter that cannot be
        negative.
CID 739656 (#7-8 of 8): Argument cannot be negative (NEGATIVE_RETURNS)
At (8): "encapsulatingSkfd" is passed to a parameter that cannot be
        negative.

Signed-off-by: Ferry Huberts <ferry.huberts@pelagic.nl>
lib/p2pd/src/NetworkInterfaces.c

index 6650134..a224462 100644 (file)
@@ -196,7 +196,6 @@ CreateInterface(const char *ifName, struct interface *olsrIntf)
   if ((olsrIntf == NULL)) {
     capturingSkfd = CreateCaptureSocket(ifName);
     if (capturingSkfd < 0) {
-      close(encapsulatingSkfd);
       free(newIf);
       return 0;
     }
@@ -215,7 +214,6 @@ CreateInterface(const char *ifName, struct interface *olsrIntf)
   if (ioctl(ioctlSkfd, SIOCGIFHWADDR, &ifr) < 0) {
     P2pdPError("ioctl(SIOCGIFHWADDR) error for interface \"%s\"", ifName);
     close(capturingSkfd);
-    close(encapsulatingSkfd);
     free(newIf);
     return 0;
   }