From a7a6dfbd14e3b65d66ed934e4ce18abfdf73ada0 Mon Sep 17 00:00:00 2001 From: Joe Marcus Clarke Date: Sun, 30 Apr 2006 07:08:43 +0000 Subject: [PATCH] Go with a different version of the previous patch so to preserve errno. Approved by: scottl (implicit) --- lib/libcam/camlib.c | 10 +++++----- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/lib/libcam/camlib.c b/lib/libcam/camlib.c index 88191ee9f4df..b2523719ed0a 100644 --- a/lib/libcam/camlib.c +++ b/lib/libcam/camlib.c @@ -465,7 +465,7 @@ static struct cam_device * cam_lookup_pass(const char *dev_name, int unit, int flags, const char *given_path, struct cam_device *device) { - int fd, rc; + int fd; union ccb ccb; char dev_path[256]; char *func_name = "cam_lookup_pass"; @@ -494,10 +494,7 @@ cam_lookup_pass(const char *dev_name, int unit, int flags, * the device name is null, if the device doesn't exist, or if the * passthrough driver isn't in the kernel. */ - rc = ioctl(fd, CAMGETPASSTHRU, &ccb); - close(fd); - - if (rc == -1) { + if (ioctl(fd, CAMGETPASSTHRU, &ccb) == -1) { char tmpstr[256]; /* @@ -517,9 +514,12 @@ cam_lookup_pass(const char *dev_name, int unit, int flags, "%s: %s%s", func_name, func_name, strerror(errno), (errno == ENOENT) ? tmpstr : ""); + close(fd); return(NULL); } + close(fd); + /* * If the ioctl returned the right status, but we got an error back * in the ccb, that means that the kernel found the device the user