diff options
author | Eric Wong <e@80x24.org> | 2022-04-18 09:50:02 +0000 |
---|---|---|
committer | Eric Wong <e@80x24.org> | 2022-04-18 21:53:46 +0000 |
commit | f9a8fba3102362e07dc27bde8e2bc7bd2a42a1ed (patch) | |
tree | 4d883e17fc680b545b6bdb26ac80f8bfc70366fa | |
parent | 6b52fe53a1f9e69a6ecfce16216a1fa180e4645f (diff) | |
download | public-inbox-f9a8fba3102362e07dc27bde8e2bc7bd2a42a1ed.tar.gz |
Since we know the space required under Linux, we can use the same initialization as the Inline::C version instead of hard-coding 256 as we do for Socket::MsgHdr.
-rw-r--r-- | lib/PublicInbox/Syscall.pm | 5 |
1 files changed, 3 insertions, 2 deletions
diff --git a/lib/PublicInbox/Syscall.pm b/lib/PublicInbox/Syscall.pm index e972aa41..cc282f9f 100644 --- a/lib/PublicInbox/Syscall.pm +++ b/lib/PublicInbox/Syscall.pm @@ -418,6 +418,7 @@ sub CMSG_ALIGN ($) { ($_[0] + SIZEOF_size_t - 1) & ~(SIZEOF_size_t - 1) } use constant CMSG_ALIGN_SIZEOF_cmsghdr => CMSG_ALIGN(SIZEOF_cmsghdr); sub CMSG_SPACE ($) { CMSG_ALIGN($_[0]) + CMSG_ALIGN_SIZEOF_cmsghdr } sub CMSG_LEN ($) { CMSG_ALIGN_SIZEOF_cmsghdr + $_[0] } +use constant msg_controllen => CMSG_SPACE(10 * SIZEOF_int) + 16; # 10 FDs if (defined($SYS_sendmsg) && defined($SYS_recvmsg)) { no warnings 'once'; @@ -457,7 +458,7 @@ no warnings 'once'; *recv_cmd4 = sub ($$$) { my ($sock, undef, $len) = @_; vec($_[1], ($len + 1) * 8, 1) = 0; - vec(my $cmsghdr = '', 256 * 8 - 1, 1) = 1; + my $cmsghdr = "\0" x msg_controllen; # 10 * sizeof(int) my $iov = pack('P'.TMPL_size_t, $_[1], $len); my $mh = pack('PL' . # msg_name, msg_namelen (socklen_t (U32)) BYTES_4_hole . # 4-byte padding on 64-bit @@ -468,7 +469,7 @@ no warnings 'once'; @BYTES_4_hole, $iov, 1, # msg_iov, msg_iovlen $cmsghdr, # msg_control - 256, # msg_controllen + msg_controllen, 0); # msg_flags my $r = syscall($SYS_recvmsg, fileno($sock), $mh, 0); return (undef) if $r < 0; # $! set |