Skip site navigation (1)Skip section navigation (2)
Date:      Mon, 26 Oct 2015 00:08:41 +0000 (UTC)
From:      Garrett Cooper <ngie@FreeBSD.org>
To:        src-committers@freebsd.org, svn-src-all@freebsd.org, svn-src-stable@freebsd.org, svn-src-stable-10@freebsd.org
Subject:   svn commit: r289972 - stable/10/lib/libcam
Message-ID:  <201510260008.t9Q08fvB044558@repo.freebsd.org>

next in thread | raw e-mail | index | archive | help
Author: ngie
Date: Mon Oct 26 00:08:40 2015
New Revision: 289972
URL: https://svnweb.freebsd.org/changeset/base/289972

Log:
  MFC r289450:
  
  Set dev->fd to -1 when calling cam_close_spec_device with a valid dev->fd
  descriptor to avoid trashing valid file descriptors that access dev->fd at a
  later point in time
  
  PR: 192671
  Submitted by: Scott Ferris <scott.ferris@isilon.com>
  Sponsored by: EMC / Isilon Storage Division

Modified:
  stable/10/lib/libcam/camlib.c
Directory Properties:
  stable/10/   (props changed)

Modified: stable/10/lib/libcam/camlib.c
==============================================================================
--- stable/10/lib/libcam/camlib.c	Mon Oct 26 00:07:11 2015	(r289971)
+++ stable/10/lib/libcam/camlib.c	Mon Oct 26 00:08:40 2015	(r289972)
@@ -676,8 +676,10 @@ cam_close_spec_device(struct cam_device 
 	if (dev == NULL)
 		return;
 
-	if (dev->fd >= 0)
+	if (dev->fd >= 0) {
 		close(dev->fd);
+		dev->fd = -1;
+	}
 }
 
 char *



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?201510260008.t9Q08fvB044558>