diff options
| author | Sami Kerola | 2017-02-11 21:23:26 +0100 |
|---|---|---|
| committer | Karel Zak | 2017-02-20 12:58:49 +0100 |
| commit | 87918040658f2fa9b1bf78f1f8f4f5c065a2e3a3 (patch) | |
| tree | 513541cfa347d7fc66e50137603489cb195c86de /sys-utils/readprofile.c | |
| parent | cfdisk: avoid use of VLA in combination with sizeof() [smatch scan] (diff) | |
| download | kernel-qcow2-util-linux-87918040658f2fa9b1bf78f1f8f4f5c065a2e3a3.tar.gz kernel-qcow2-util-linux-87918040658f2fa9b1bf78f1f8f4f5c065a2e3a3.tar.xz kernel-qcow2-util-linux-87918040658f2fa9b1bf78f1f8f4f5c065a2e3a3.zip | |
misc: do not use plain 0 as NULL [smatch scan]
text-utils/tailf.c:69:21: warning: Using plain integer as NULL pointer
Since many 'struct option' has used zero as NULL make them more readable in
same go by reindenting, and using named argument requirements.
Reference: https://lwn.net/Articles/93577/
Signed-off-by: Sami Kerola <kerolasa@iki.fi>
Diffstat (limited to 'sys-utils/readprofile.c')
| -rw-r--r-- | sys-utils/readprofile.c | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/sys-utils/readprofile.c b/sys-utils/readprofile.c index 0170a7240..f18fa9bf2 100644 --- a/sys-utils/readprofile.c +++ b/sys-utils/readprofile.c @@ -132,7 +132,7 @@ int main(int argc, char **argv) { FILE *map; int proFd; - char *mapFile, *proFile, *mult = 0; + char *mapFile, *proFile, *mult = NULL; size_t len = 0, indx = 1; unsigned long long add0 = 0; unsigned int step; @@ -162,7 +162,7 @@ int main(int argc, char **argv) {"no-auto", no_argument, NULL, 'n'}, {"version", no_argument, NULL, 'V'}, {"help", no_argument, NULL, 'h'}, - {NULL, 0, 0, 0} + {NULL, 0, NULL, 0} }; #define next (current^1) @@ -224,7 +224,7 @@ int main(int argc, char **argv) * write is not sizeof(int), the multiplier is not * changed. */ if (mult) { - multiplier = strtoul(mult, 0, 10); + multiplier = strtoul(mult, NULL, 10); to_write = sizeof(int); } else { multiplier = 0; |