diff options
| author | Ruediger Meier | 2017-06-21 03:01:30 +0200 |
|---|---|---|
| committer | Ruediger Meier | 2017-06-22 02:56:57 +0200 |
| commit | 0caad87fdb87789f40942e14f81539af84c2a55b (patch) | |
| tree | d37e7606008f229fc930d9387f27c655e5253702 /login-utils/lslogins.c | |
| parent | uuidd: remove unused define (diff) | |
| download | kernel-qcow2-util-linux-0caad87fdb87789f40942e14f81539af84c2a55b.tar.gz kernel-qcow2-util-linux-0caad87fdb87789f40942e14f81539af84c2a55b.tar.xz kernel-qcow2-util-linux-0caad87fdb87789f40942e14f81539af84c2a55b.zip | |
misc: remove superfluous null pointer checks for optarg
This is only needed for optional arguments.
Signed-off-by: Ruediger Meier <ruediger.meier@ga-group.nl>
Diffstat (limited to 'login-utils/lslogins.c')
| -rw-r--r-- | login-utils/lslogins.c | 15 |
1 files changed, 6 insertions, 9 deletions
diff --git a/login-utils/lslogins.c b/login-utils/lslogins.c index ab04c10bb..1e89d2703 100644 --- a/login-utils/lslogins.c +++ b/login-utils/lslogins.c @@ -1381,15 +1381,12 @@ int main(int argc, char *argv[]) outmode = OUT_NEWLINE; break; case 'o': - if (optarg) { - if (*optarg == '=') - optarg++; - ncolumns = string_to_idarray(optarg, - columns, ARRAY_SIZE(columns), - column_name_to_id); - if (ncolumns < 0) - return EXIT_FAILURE; - } + if (*optarg == '=') + optarg++; + ncolumns = string_to_idarray(optarg, columns, + ARRAY_SIZE(columns), column_name_to_id); + if (ncolumns < 0) + return EXIT_FAILURE; opt_o = 1; break; case 'r': |