svn commit: r297961 - head/usr.sbin/rmt
Marcelo Araujo
araujo at FreeBSD.org
Thu Apr 14 12:25:02 UTC 2016
Author: araujo
Date: Thu Apr 14 12:25:00 2016
New Revision: 297961
URL: https://svnweb.freebsd.org/changeset/base/297961
Log:
Use NULL instead of 0 for pointers.
fopen(3) will return NULL in case it can't open the STREAM.
The malloc will return a pointer to the allocated memory if successful,
otherwise a NULL pointer is returned.
Also add an extra DEBUG1 to print out the error to open a file.
Reviewed by: ed
Differential Revision: https://svnweb.freebsd.org/changeset/base/297959
Modified:
head/usr.sbin/rmt/rmt.c
Modified: head/usr.sbin/rmt/rmt.c
==============================================================================
--- head/usr.sbin/rmt/rmt.c Thu Apr 14 11:45:52 2016 (r297960)
+++ head/usr.sbin/rmt/rmt.c Thu Apr 14 12:25:00 2016 (r297961)
@@ -84,8 +84,10 @@ main(int argc, char **argv)
argc--, argv++;
if (argc > 0) {
debug = fopen(*argv, "w");
- if (debug == 0)
+ if (debug == NULL) {
+ DEBUG1("rmtd: error to open %s\n", *argv);
exit(1);
+ }
(void)setbuf(debug, (char *)0);
}
top:
@@ -226,10 +228,10 @@ checkbuf(char *rec, int size)
if (size <= maxrecsize)
return (rec);
- if (rec != 0)
+ if (rec != NULL)
free(rec);
rec = malloc(size);
- if (rec == 0) {
+ if (rec == NULL) {
DEBUG("rmtd: cannot allocate buffer space\n");
exit(4);
}
More information about the svn-src-head
mailing list