diff options
author | Mark Johnston <markj@FreeBSD.org> | 2023-04-26 21:19:28 +0000 |
---|---|---|
committer | Mark Johnston <markj@FreeBSD.org> | 2023-05-26 19:38:08 +0000 |
commit | ca4c7859009549975229d854888231e48938035e (patch) | |
tree | 91a3d74a8f5626a61fd239f078891f01023f4a3d | |
parent | 44866dfb44178c86b6f380eb713a8f07c3d3b566 (diff) | |
download | src-ca4c7859009549975229d854888231e48938035e.tar.gz src-ca4c7859009549975229d854888231e48938035e.zip |
nexus: Consistently return a pointer in failure paths
No functional change intended.
MFC after: 1 week
-rw-r--r-- | sys/arm/arm/nexus.c | 2 | ||||
-rw-r--r-- | sys/x86/x86/nexus.c | 4 |
2 files changed, 3 insertions, 3 deletions
diff --git a/sys/arm/arm/nexus.c b/sys/arm/arm/nexus.c index 48e766b0cc62..430373d3679f 100644 --- a/sys/arm/arm/nexus.c +++ b/sys/arm/arm/nexus.c @@ -256,7 +256,7 @@ nexus_alloc_resource(device_t bus, device_t child, int type, int *rid, if (needactivate) { if (bus_activate_resource(child, type, *rid, rv)) { rman_release_resource(rv); - return (0); + return (NULL); } } diff --git a/sys/x86/x86/nexus.c b/sys/x86/x86/nexus.c index d02d081de1e8..332b1527efbd 100644 --- a/sys/x86/x86/nexus.c +++ b/sys/x86/x86/nexus.c @@ -388,13 +388,13 @@ nexus_alloc_resource(device_t bus, device_t child, int type, int *rid, rv = rman_reserve_resource(rm, start, end, count, flags, child); if (rv == NULL) - return (0); + return (NULL); rman_set_rid(rv, *rid); if (needactivate) { if (bus_activate_resource(child, type, *rid, rv)) { rman_release_resource(rv); - return (0); + return (NULL); } } |