From: Jon Bernard <jbernard@debian.org>
Date: Tue, 9 Mar 2010 21:10:39 +0000 (-0500)
Subject: Use memset to initialize sigaction struct
X-Git-Tag: v0.36.2~10
X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=8831ca690cc970716953f73b206f258ff198711f;p=thirdparty%2Flibcgroup.git

Use memset to initialize sigaction struct

The sa_restorer field is not present on some architectures, like alpha.
By using memset, the structure is initialized appropriately regardless of
struct contents.

Signed-off-by: Jon Bernard <jbernard@debian.org>
Acked-by: Balbir Singh <balbir@linux.vnet.ibm.com>
Signed-off-by: Dhaval Giani <dhaval.giani@gmail.com>
---

diff --git a/src/daemon/cgrulesengd.c b/src/daemon/cgrulesengd.c
index 65f6c03c..c39aacc2 100644
--- a/src/daemon/cgrulesengd.c
+++ b/src/daemon/cgrulesengd.c
@@ -1034,9 +1034,8 @@ int main(int argc, char *argv[])
 	 * Set up the signal handler to reload the cached rules upon reception
 	 * of a SIGUSR2 signal.
 	 */
+	memset(&sa, 0, sizeof(sa));
 	sa.sa_handler = &cgre_flash_rules;
-	sa.sa_flags = 0;
-	sa.sa_restorer = NULL;
 	sigemptyset(&sa.sa_mask);
 	if ((ret = sigaction(SIGUSR2, &sa, NULL))) {
 		flog(LOG_ERR, "Failed to set up signal handler for SIGUSR2."