From ef554f759ea83834b62de116498300bba47c9ec4 Mon Sep 17 00:00:00 2001 From: ngie Date: Thu, 23 Mar 2017 04:50:44 +0000 Subject: 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. --- lib/libkvm/kvm.c | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) (limited to 'lib/libkvm') 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)); -- cgit v1.1