From: Kay Sievers <kay.sievers@vrfy.org>
Date: Tue, 12 Apr 2011 23:47:56 +0000 (+0200)
Subject: trivial cleanups
X-Git-Tag: 174~197
X-Git-Url: http://www.chiark.greenend.org.uk/ucgi/~ianmdlvl/git?a=commitdiff_plain;ds=sidebyside;h=e48e2912023b5600d291904b0f7b0017387e8cb2;p=elogind.git

trivial cleanups
---

diff --git a/extras/ata_id/ata_id.c b/extras/ata_id/ata_id.c
index 594cfb201..cb4a1a1f0 100644
--- a/extras/ata_id/ata_id.c
+++ b/extras/ata_id/ata_id.c
@@ -486,8 +486,6 @@ int main(int argc, char *argv[])
 			printf("Usage: ata_id [--export] [--help] <device>\n"
 			       "  --export    print values as environment keys\n"
 			       "  --help      print this help text\n\n");
-		default:
-			rc = 1;
 			goto exit;
 		}
 	}
diff --git a/extras/firmware/firmware.c b/extras/firmware/firmware.c
index f2df76c8c..36ce79054 100644
--- a/extras/firmware/firmware.c
+++ b/extras/firmware/firmware.c
@@ -43,7 +43,7 @@ static bool set_loading(struct udev *udev, char *loadpath, const char *state)
 static bool copy_firmware(struct udev *udev, const char *source, const char *target, size_t size)
 {
 	char *buf;
-	FILE *fsource, *ftarget;
+	FILE *fsource = NULL, *ftarget = NULL;
 	bool ret = false;
 
 	buf = malloc(size);
@@ -112,8 +112,6 @@ int main(int argc, char **argv)
 			break;
 		case 'h':
 			printf("Usage: firmware --firmware=<fwfile> --devpath=<path> [--help]\n\n");
-		default:
-			rc = 1;
 			goto exit;
 		}
 	}
diff --git a/extras/hid2hci/hid2hci.c b/extras/hid2hci/hid2hci.c
index 839c4fbce..98e60d9ac 100644
--- a/extras/hid2hci/hid2hci.c
+++ b/extras/hid2hci/hid2hci.c
@@ -243,7 +243,7 @@ int main(int argc, char *argv[])
 	for (;;) {
 		int option;
 
-		option = getopt_long(argc, argv, "m:p:M:qh", options, NULL);
+		option = getopt_long(argc, argv, "m:p:M:h", options, NULL);
 		if (option == -1)
 			break;
 
@@ -277,8 +277,6 @@ int main(int argc, char *argv[])
 			break;
 		case 'h':
 			usage(NULL);
-		default:
-			exit(1);
 		}
 	}
 
diff --git a/extras/path_id/path_id.c b/extras/path_id/path_id.c
index c19bfd0f8..db13eeba2 100644
--- a/extras/path_id/path_id.c
+++ b/extras/path_id/path_id.c
@@ -388,8 +388,6 @@ int main(int argc, char **argv)
 			printf("Usage: path_id [--debug] [--help] <devpath>\n"
 			       "  --debug    print debug information\n"
 			       "  --help      print this help text\n\n");
-		default:
-			rc = 1;
 			goto exit;
 		}
 	}
diff --git a/extras/udev-acl/udev-acl.c b/extras/udev-acl/udev-acl.c
index 31e9991a5..2dd3e6cac 100644
--- a/extras/udev-acl/udev-acl.c
+++ b/extras/udev-acl/udev-acl.c
@@ -364,7 +364,6 @@ int main (int argc, char* argv[])
 			break;
 		case 'h':
 			printf("Usage: udev-acl --action=ACTION [--device=DEVICEFILE] [--user=UID]\n\n");
-		default:
 			goto out;
 		}
 	}
diff --git a/extras/usb_id/usb_id.c b/extras/usb_id/usb_id.c
index fabd09283..3a7a0ffd4 100644
--- a/extras/usb_id/usb_id.c
+++ b/extras/usb_id/usb_id.c
@@ -503,8 +503,6 @@ int main(int argc, char **argv)
 			       "  --num-info  use numerical values\n"
 			       "  --export    print values as environment keys\n"
 			       "  --help      print this help text\n\n");
-		default:
-			retval = 1;
 			goto exit;
 		}
 	}
diff --git a/libudev/libudev-util-private.c b/libudev/libudev-util-private.c
index 19f979eea..073f7e2df 100644
--- a/libudev/libudev-util-private.c
+++ b/libudev/libudev-util-private.c
@@ -134,7 +134,7 @@ int util_unlink_secure(struct udev *udev, const char *filename)
 uid_t util_lookup_user(struct udev *udev, const char *user)
 {
 	char *endptr;
-	int buflen = sysconf(_SC_GETPW_R_SIZE_MAX);
+	size_t buflen = sysconf(_SC_GETPW_R_SIZE_MAX);
 	char buf[buflen];
 	struct passwd pwbuf;
 	struct passwd *pw;
@@ -159,7 +159,7 @@ uid_t util_lookup_user(struct udev *udev, const char *user)
 gid_t util_lookup_group(struct udev *udev, const char *group)
 {
 	char *endptr;
-	int buflen = sysconf(_SC_GETGR_R_SIZE_MAX);
+	size_t buflen = sysconf(_SC_GETGR_R_SIZE_MAX);
 	char *buf;
 	struct group grbuf;
 	struct group *gr;
diff --git a/udev/udevadm-monitor.c b/udev/udevadm-monitor.c
index 85252bb7d..a587e8cc0 100644
--- a/udev/udevadm-monitor.c
+++ b/udev/udevadm-monitor.c
@@ -133,7 +133,6 @@ int udevadm_monitor(struct udev *udev, int argc, char *argv[])
 			       "  --subsystem-match=<subsystem[/devtype]> filter events by subsystem\n"
 			       "  --tag-match=<tag>                       filter events by tag\n"
 			       "  --help\n\n");
-		default:
 			goto out;
 		}
 	}
diff --git a/udev/udevd.c b/udev/udevd.c
index c785b2075..8610c6c06 100644
--- a/udev/udevd.c
+++ b/udev/udevd.c
@@ -744,9 +744,11 @@ static int handle_inotify(struct udev *udev)
 				info(udev, "device %s closed, synthesising 'change'\n", udev_device_get_devnode(dev));
 				util_strscpyl(filename, sizeof(filename), udev_device_get_syspath(dev), "/uevent", NULL);
 				fd = open(filename, O_WRONLY);
-				if (fd < 0 || write(fd, "change", 6) < 0)
-					info(udev, "error writing uevent: %m\n");
-				close(fd);
+				if (fd >= 0) {
+					if (write(fd, "change", 6) < 0)
+						info(udev, "error writing uevent: %m\n");
+					close(fd);
+				}
 			}
 			if (ev->mask & IN_IGNORED)
 				udev_watch_end(udev, dev);