Skip site navigation (1)Skip section navigation (2)
Date:      Mon, 4 Jan 2016 07:11:15 +0000 (UTC)
From:      Garrett Cooper <ngie@FreeBSD.org>
To:        src-committers@freebsd.org, svn-src-all@freebsd.org, svn-src-stable@freebsd.org, svn-src-stable-10@freebsd.org
Subject:   svn commit: r293145 - stable/10/tools/regression/sockets/unix_passfd
Message-ID:  <201601040711.u047BFV1085064@repo.freebsd.org>

next in thread | raw e-mail | index | archive | help
Author: ngie
Date: Mon Jan  4 07:11:15 2016
New Revision: 293145
URL: https://svnweb.freebsd.org/changeset/base/293145

Log:
  MFC r292814:
  
  - Explicitly initialize ch to 0
  - Delete some spurious whitespace
  - Use calloc instead of malloc in the last test to ensure that
    sendspace is properly zero'ed out
  
  Submitted by: markj

Modified:
  stable/10/tools/regression/sockets/unix_passfd/unix_passfd.c
Directory Properties:
  stable/10/   (props changed)

Modified: stable/10/tools/regression/sockets/unix_passfd/unix_passfd.c
==============================================================================
--- stable/10/tools/regression/sockets/unix_passfd/unix_passfd.c	Mon Jan  4 07:09:29 2016	(r293144)
+++ stable/10/tools/regression/sockets/unix_passfd/unix_passfd.c	Mon Jan  4 07:11:15 2016	(r293145)
@@ -1,5 +1,6 @@
 /*-
  * Copyright (c) 2005 Robert N. M. Watson
+ * Copyright (c) 2015 Mark Johnston
  * All rights reserved.
  *
  * Redistribution and use in source and binary forms, with or without
@@ -146,7 +147,7 @@ sendfd_payload(const char *test, int soc
 static void
 sendfd(const char *test, int sockfd, int sendfd)
 {
-	char ch;
+	char ch = 0;
 
 	return (sendfd_payload(test, sockfd, sendfd, &ch, sizeof(ch)));
 }
@@ -199,7 +200,7 @@ recvfd_payload(const char *test, int soc
 static void
 recvfd(const char *test, int sockfd, int *recvfd)
 {
-	char ch;
+	char ch = 0;
 
 	return (recvfd_payload(test, sockfd, recvfd, &ch, sizeof(ch)));
 }
@@ -369,8 +370,8 @@ main(void)
 			err(-1, "%s: sysctlbyname(net.local.stream.sendspace)",
 			    test);
 
-		if ((buf = malloc(sendspace)) == NULL)
-			err(-1, "%s: malloc", test);
+		if ((buf = calloc(1, sendspace)) == NULL)
+			err(-1, "%s: calloc", test);
 
 		domainsocketpair(test, fd);
 		if (setsockopt(fd[1], 0, LOCAL_CREDS, &on, sizeof(on)) < 0)
@@ -384,6 +385,6 @@ main(void)
 	}
 
 	printf("%s passed\n", test);
-	
+
 	return (0);
 }



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?201601040711.u047BFV1085064>