VesperOS/shadow
6
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

lib/sgetspent.c: sgetspent(): Explicitly use an empty string literal

Browse Source 
cp can only be an empty string literal in that conditional.  Use a
string literal to be more explicit.

Signed-off-by: Alejandro Colomar <alx@kernel.org>
This commit is contained in:
Alejandro Colomar
2024-01-15 18:23:17 +01:00
committed by Serge Hallyn
parent 93151689c0
commit 34b113baba
1 changed files with 2 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files

5
lib/sgetspent.c
View File

@@ -73,9 +73,8 @@ sgetspent(const char *string)
}
}
if (i == (FIELDS - 1)) {
fields[i++] = cp;
}
if (i == (FIELDS - 1))
fields[i++] = "";
if ( ((NULL != cp) && ('\0' != *cp)) ||
((i != FIELDS) && (i != OFIELDS)) ) {