git: b7f127b6e5f9 - main - ipfilter: Save a word of stack space
- Go to: [ bottom of page ] [ top of archives ] [ this month ]
Date: Sat, 04 Dec 2021 14:34:33 UTC
The branch main has been updated by cy:
URL: https://cgit.FreeBSD.org/src/commit/?id=b7f127b6e5f9febff895da9230c3269dfcd971c2
commit b7f127b6e5f9febff895da9230c3269dfcd971c2
Author: Cy Schubert <cy@FreeBSD.org>
AuthorDate: 2021-10-04 18:36:15 +0000
Commit: Cy Schubert <cy@FreeBSD.org>
CommitDate: 2021-12-04 14:33:12 +0000
ipfilter: Save a word of stack space
Rather than save the return code into an intermediate variable, which BTW
is optimized out anyway, explicitly remove the return code from the stack.
MFC after: 3 days
---
sys/contrib/ipfilter/netinet/ip_ftp_pxy.c | 6 ++----
1 file changed, 2 insertions(+), 4 deletions(-)
diff --git a/sys/contrib/ipfilter/netinet/ip_ftp_pxy.c b/sys/contrib/ipfilter/netinet/ip_ftp_pxy.c
index f4b40f3601d9..0079e884a8b8 100644
--- a/sys/contrib/ipfilter/netinet/ip_ftp_pxy.c
+++ b/sys/contrib/ipfilter/netinet/ip_ftp_pxy.c
@@ -1258,15 +1258,13 @@ ipf_p_ftp_valid(softf, ftp, side, buf, len)
size_t len;
{
ftpside_t *ftps;
- int ret;
ftps = &ftp->ftp_side[side];
if (side == 0)
- ret = ipf_p_ftp_client_valid(softf, ftps, buf, len);
+ return(ipf_p_ftp_client_valid(softf, ftps, buf, len));
else
- ret = ipf_p_ftp_server_valid(softf, ftps, buf, len);
- return ret;
+ return(ipf_p_ftp_server_valid(softf, ftps, buf, len));
}