diff options
author | Konstantin Belousov <kib@FreeBSD.org> | 2021-10-21 17:40:24 +0000 |
---|---|---|
committer | Konstantin Belousov <kib@FreeBSD.org> | 2021-10-21 18:40:46 +0000 |
commit | 6776747a0e0b1c0e3fb916685ac2180ca4ce2f3d (patch) | |
tree | 129dce099a9b21a002e6db56526c95fbf2c9720f /sys/kern/subr_firmware.c | |
parent | 993446638ce701109d29c520b85d7ca31b5c5fe8 (diff) | |
download | src-6776747a0e0b1c0e3fb916685ac2180ca4ce2f3d.tar.gz src-6776747a0e0b1c0e3fb916685ac2180ca4ce2f3d.zip |
subr_firmware.c::unloadentry(): remote write-only variable
The function ignores result returned by linker_release_module().
The FW_UNLOAD flag on the file is cleared, so even on error it would
not be tried again.
Sponsored by: The FreeBSD Foundation
MFC after: 1 week
Diffstat (limited to 'sys/kern/subr_firmware.c')
-rw-r--r-- | sys/kern/subr_firmware.c | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/sys/kern/subr_firmware.c b/sys/kern/subr_firmware.c index 0465f2a88483..aafb2a2ba265 100644 --- a/sys/kern/subr_firmware.c +++ b/sys/kern/subr_firmware.c @@ -412,7 +412,6 @@ static void unloadentry(void *unused1, int unused2) { struct priv_fw *fp; - int err; mtx_lock(&firmware_mtx); restart: @@ -434,7 +433,7 @@ restart: * on unload to actually free the entry. */ mtx_unlock(&firmware_mtx); - err = linker_release_module(NULL, NULL, fp->file); + (void)linker_release_module(NULL, NULL, fp->file); mtx_lock(&firmware_mtx); /* |