tools: lxc-unshare: fix wrong coding rules

src/lxc/tools/arguments.c

@@ -338,4 +338,4 @@ bool lxc_setup_shared_ns(struct lxc_arguments *args, struct lxc_container *c)
 	}
 
 	return true;
-}
\ No newline at end of file
+}

src/lxc/tools/lxc_unshare.c

@@ -71,7 +71,7 @@ static int mount_fs(const char *source, const char *target, const char *type);
 static void lxc_setup_fs(void);
 static int do_start(void *arg);
 
-static struct my_iflist *tmpif, *my_iflist = NULL;
+static struct my_iflist *tmpif, *my_iflist;
 
 static const struct option my_longopts[] = {
 	{"namespaces", required_argument, 0, 's'},
@@ -128,7 +128,8 @@ static int my_parser(struct lxc_arguments *args, int c, char *arg)
 		args->want_hostname = arg;
 		break;
 	case 'i':
-		if (!(tmpif = malloc(sizeof(*tmpif)))) {
+		tmpif = malloc(sizeof(*tmpif));
+		if (!tmpif) {
 			SYSERROR("Failed to malloc()");
 			return -1;
 		}
@@ -327,7 +328,7 @@ int main(int argc, char *argv[])
 			exit(EXIT_FAILURE);
 	}
 
-	if (*my_args.argv == NULL) {
+	if (!*my_args.argv) {
 		ERROR("A command to execute in the new namespace is required");
 		exit(EXIT_FAILURE);
 	}
@@ -443,4 +444,4 @@ int main(int argc, char *argv[])
 
 	/* Call exit() directly on this function because it retuns an exit code. */
 	exit(EXIT_SUCCESS);
-}
\ No newline at end of file
+}