report error if statefd parm is not a digit

src/lxc/arguments.c

@@ -230,3 +230,23 @@ extern char **lxc_arguments_dup(const char *file, struct lxc_arguments *args)
 
 	return argv;
 }
+
+int lxc_arguments_str_to_int(struct lxc_arguments *args, const char *str)
+{
+	long val;
+	char *endptr;
+
+	errno = 0;
+	val = strtol(str, &endptr, 10);
+	if (errno) {
+		lxc_error(args, "invalid statefd '%s' : %m", str);
+		return -1;
+	}
+
+	if (*endptr) {
+		lxc_error(args, "invalid digit for statefd '%s'", str);
+		return -1;
+	}
+
+	return (int)val;
+}

src/lxc/arguments.h

@@ -78,6 +78,7 @@ extern int lxc_arguments_parse(struct lxc_arguments *args,
 			       int argc, char *const argv[]);
 
 extern char **lxc_arguments_dup(const char *file, struct lxc_arguments *args);
+extern int lxc_arguments_str_to_int(struct lxc_arguments *args, const char *str);
 
 extern const char *lxc_strerror(int errnum);
 

src/lxc/lxc_checkpoint.c

@@ -60,15 +60,12 @@ static int my_parser(struct lxc_arguments* args, int c, char* arg)
 	case 'p': args->flags = LXC_FLAG_PAUSE; break;
 	case 'S': args->statefile = arg; break;
 	case 'd': {
-			long val;
-			errno = 0;
-			val = strtol(arg, (char **)NULL, 10);
-			if (errno) {
-				lxc_error(args, "invalid statefd '%s' : %m\n",
-					  arg);
+			int fd;
+			fd = lxc_arguments_str_to_int(args, arg);
+			if (fd < 0)
 				return -1;
-			}
-			args->statefd = (int)val;
+
+			args->statefd = fd;
 			break;
 		}
 	}

src/lxc/lxc_restart.c

@@ -63,15 +63,12 @@ static int my_parser(struct lxc_arguments* args, int c, char* arg)
 	case 'p': args->flags = LXC_FLAG_PAUSE; break;
 	case 's': return lxc_config_define_add(&defines, arg);
 	case 'd': {
-			long val;
-			errno = 0;
-			val = strtol(arg, (char **)NULL, 10);
-			if (errno) {
-				lxc_error(args, "invalid statefd '%s' : %m\n",
-					  arg);
+			int fd;
+			fd = lxc_arguments_str_to_int(args, arg);
+			if (fd < 0)
 				return -1;
-			}
-			args->statefd = (int)val;
+
+			args->statefd = fd;
 			break;
 		}
 	}