Author: markj
Date: Wed Aug  1 19:45:04 2018
New Revision: 337059
URL: https://svnweb.freebsd.org/changeset/base/337059

Log:
  Fix some nits in the unix_passfd tests.
  
  - Remove return statements in functions with a void return type.
  - Allocate enough space for the SCM_CREDS and SCM_RIGHTS messages
    received in the rights_creds_payload test.
  
  MFC after:    1 week
  Sponsored by: The FreeBSD Foundation

Modified:
  head/tests/sys/kern/unix_passfd_test.c

Modified: head/tests/sys/kern/unix_passfd_test.c
==============================================================================
--- head/tests/sys/kern/unix_passfd_test.c      Wed Aug  1 19:25:12 2018        
(r337058)
+++ head/tests/sys/kern/unix_passfd_test.c      Wed Aug  1 19:45:04 2018        
(r337059)
@@ -165,14 +165,15 @@ sendfd(int sockfd, int send_fd)
 {
        char ch = 0;
 
-       return (sendfd_payload(sockfd, send_fd, &ch, sizeof(ch)));
+       sendfd_payload(sockfd, send_fd, &ch, sizeof(ch));
 }
 
 static void
 recvfd_payload(int sockfd, int *recv_fd, void *buf, size_t buflen)
 {
        struct cmsghdr *cmsghdr;
-       char message[CMSG_SPACE(SOCKCREDSIZE(CMGROUP_MAX)) + sizeof(int)];
+       char message[CMSG_SPACE(SOCKCREDSIZE(CMGROUP_MAX)) +
+           CMSG_SPACE(sizeof(int))];
        struct msghdr msghdr;
        struct iovec iovec;
        ssize_t len;
@@ -214,7 +215,7 @@ recvfd(int sockfd, int *recv_fd)
 {
        char ch = 0;
 
-       return (recvfd_payload(sockfd, recv_fd, &ch, sizeof(ch)));
+       recvfd_payload(sockfd, recv_fd, &ch, sizeof(ch));
 }
 
 /*
_______________________________________________
[email protected] mailing list
https://lists.freebsd.org/mailman/listinfo/svn-src-all
To unsubscribe, send any mail to "[email protected]"

Reply via email to