s-s-d: Use a new xstrdup that never returns NULL instead of strdup

ChangeLog

@@ -1,3 +1,8 @@
+2009-01-14  Guillem Jover  <guillem@debian.org>
+
+	* utils/start-stop-daemon.c (xstrdup): New function.
+	(parse_options): Use xstrdup instead of strdup.
+
 2009-01-10  Guillem Jover  <guillem@debian.org>
 
 	* utils/start-stop-daemon.c (main): If changeuser is a uid, call

utils/start-stop-daemon.c

@@ -199,6 +199,17 @@ xmalloc(int size)
 	fatal("malloc(%d) failed", size);
 }
 
+static char *
+xstrdup(const char *str)
+{
+	char *new_str;
+
+	new_str = strdup(str);
+	if (new_str)
+		return new_str;
+	fatal("strdup(%s) failed", str);
+}
+
 static void
 xgettimeofday(struct timeval *tv)
 {
@@ -584,7 +595,7 @@ parse_options(int argc, char * const *argv)
 		case 'c':  /* --chuid <username>|<uid> */
 			/* we copy the string just in case we need the
 			 * argument later. */
-			changeuser = strdup(optarg);
+			changeuser = xstrdup(optarg);
 			changeuser = strtok(changeuser, ":");
 			changegroup = strtok(NULL, ":");
 			break;