diff options
| author | rwatson <rwatson@FreeBSD.org> | 2008-12-16 15:05:35 +0000 |
|---|---|---|
| committer | rwatson <rwatson@FreeBSD.org> | 2008-12-16 15:05:35 +0000 |
| commit | 662640d1ee6a45b933d10e23b9e3cbb9d5bf0d62 (patch) | |
| tree | 12d3b1063e176329f5dc8e65f7f1193d112d9ad7 /sys/netinet/ip_fw_pfil.c | |
| parent | f702951b49e8263ed5b1962583adfa984f602e83 (diff) | |
| download | FreeBSD-src-662640d1ee6a45b933d10e23b9e3cbb9d5bf0d62.zip FreeBSD-src-662640d1ee6a45b933d10e23b9e3cbb9d5bf0d62.tar.gz | |
IPFW's pfil hook/unhook code ignores the return values of pfil_add_hook()
and pfil_remove_hook(), so cast them to (void).
MFC after: pretty soon
Diffstat (limited to 'sys/netinet/ip_fw_pfil.c')
| -rw-r--r-- | sys/netinet/ip_fw_pfil.c | 24 |
1 files changed, 16 insertions, 8 deletions
diff --git a/sys/netinet/ip_fw_pfil.c b/sys/netinet/ip_fw_pfil.c index 952574d..694f8ab 100644 --- a/sys/netinet/ip_fw_pfil.c +++ b/sys/netinet/ip_fw_pfil.c @@ -435,8 +435,10 @@ ipfw_hook(void) if (pfh_inet == NULL) return ENOENT; - pfil_add_hook(ipfw_check_in, NULL, PFIL_IN | PFIL_WAITOK, pfh_inet); - pfil_add_hook(ipfw_check_out, NULL, PFIL_OUT | PFIL_WAITOK, pfh_inet); + (void)pfil_add_hook(ipfw_check_in, NULL, PFIL_IN | PFIL_WAITOK, + pfh_inet); + (void)pfil_add_hook(ipfw_check_out, NULL, PFIL_OUT | PFIL_WAITOK, + pfh_inet); return 0; } @@ -450,8 +452,10 @@ ipfw_unhook(void) if (pfh_inet == NULL) return ENOENT; - pfil_remove_hook(ipfw_check_in, NULL, PFIL_IN | PFIL_WAITOK, pfh_inet); - pfil_remove_hook(ipfw_check_out, NULL, PFIL_OUT | PFIL_WAITOK, pfh_inet); + (void)pfil_remove_hook(ipfw_check_in, NULL, PFIL_IN | PFIL_WAITOK, + pfh_inet); + (void)pfil_remove_hook(ipfw_check_out, NULL, PFIL_OUT | PFIL_WAITOK, + pfh_inet); return 0; } @@ -466,8 +470,10 @@ ipfw6_hook(void) if (pfh_inet6 == NULL) return ENOENT; - pfil_add_hook(ipfw_check_in, NULL, PFIL_IN | PFIL_WAITOK, pfh_inet6); - pfil_add_hook(ipfw_check_out, NULL, PFIL_OUT | PFIL_WAITOK, pfh_inet6); + (void)pfil_add_hook(ipfw_check_in, NULL, PFIL_IN | PFIL_WAITOK, + pfh_inet6); + (void)pfil_add_hook(ipfw_check_out, NULL, PFIL_OUT | PFIL_WAITOK, + pfh_inet6); return 0; } @@ -481,8 +487,10 @@ ipfw6_unhook(void) if (pfh_inet6 == NULL) return ENOENT; - pfil_remove_hook(ipfw_check_in, NULL, PFIL_IN | PFIL_WAITOK, pfh_inet6); - pfil_remove_hook(ipfw_check_out, NULL, PFIL_OUT | PFIL_WAITOK, pfh_inet6); + (void)pfil_remove_hook(ipfw_check_in, NULL, PFIL_IN | PFIL_WAITOK, + pfh_inet6); + (void)pfil_remove_hook(ipfw_check_out, NULL, PFIL_OUT | PFIL_WAITOK, + pfh_inet6); return 0; } |
