diff options
author | ngie <ngie@FreeBSD.org> | 2017-03-23 04:50:44 +0000 |
---|---|---|
committer | ngie <ngie@FreeBSD.org> | 2017-03-23 04:50:44 +0000 |
commit | ef554f759ea83834b62de116498300bba47c9ec4 (patch) | |
tree | c9cafd2b3c577df8befa91ce38c87e820c4c7516 | |
parent | 5b972b6c9ce470690a1a551e96cacdb85a4873fb (diff) | |
download | FreeBSD-src-ef554f759ea83834b62de116498300bba47c9ec4.zip FreeBSD-src-ef554f759ea83834b62de116498300bba47c9ec4.tar.gz |
MFC r315360:
Return NULL instead of 0 on failure in _kvm_open, kvm_open{,2,files}
This is being done for the following reasons:
- kvm_open(3), etc says they will return NULL.
- NULL by definition is (void*)0 per POSIX, but can be redefined,
depending on the compiler, etc.
-rw-r--r-- | lib/libkvm/kvm.c | 8 |
1 files changed, 4 insertions, 4 deletions
diff --git a/lib/libkvm/kvm.c b/lib/libkvm/kvm.c index c8c34f2..0de0dd6 100644 --- a/lib/libkvm/kvm.c +++ b/lib/libkvm/kvm.c @@ -479,7 +479,7 @@ failed: if (errout != NULL) strlcpy(errout, kd->errbuf, _POSIX2_LINE_MAX); (void)kvm_close(kd); - return (0); + return (NULL); } kvm_t * @@ -492,7 +492,7 @@ kvm_openfiles(const char *uf, const char *mf, const char *sf __unused, int flag, if (errout != NULL) (void)strlcpy(errout, strerror(errno), _POSIX2_LINE_MAX); - return (0); + return (NULL); } return (_kvm_open(kd, uf, mf, flag, errout)); } @@ -507,7 +507,7 @@ kvm_open(const char *uf, const char *mf, const char *sf __unused, int flag, if (errstr != NULL) (void)fprintf(stderr, "%s: %s\n", errstr, strerror(errno)); - return (0); + return (NULL); } kd->program = errstr; return (_kvm_open(kd, uf, mf, flag, NULL)); @@ -523,7 +523,7 @@ kvm_open2(const char *uf, const char *mf, int flag, char *errout, if (errout != NULL) (void)strlcpy(errout, strerror(errno), _POSIX2_LINE_MAX); - return (0); + return (NULL); } kd->resolve_symbol = resolver; return (_kvm_open(kd, uf, mf, flag, errout)); |