[asterisk-commits] build: Fix building with newer GCC. (asterisk[11])

SVN commits to the Asterisk project asterisk-commits at lists.digium.com
Fri Dec 4 11:34:27 CST 2015


Matt Jordan has submitted this change and it was merged.

Change subject: build: Fix building with newer GCC.
......................................................................


build: Fix building with newer GCC.

Newer GCC is upset that the features flags are uninitialized.
Zero them out so it is less upset.

Change-Id: I05162699e1b36bce4145f58a63b47ad19c6975ac
---
M main/features.c
1 file changed, 1 insertion(+), 3 deletions(-)

Approvals:
  Richard Mudgett: Looks good to me, but someone else must approve
  Anonymous Coward #1000019: Verified
  Matt Jordan: Looks good to me, approved
  Walter Doekes: Looks good to me, but someone else must approve



diff --git a/main/features.c b/main/features.c
index c0f4b8d..4da60bc 100644
--- a/main/features.c
+++ b/main/features.c
@@ -3722,14 +3722,12 @@
 
 	char dynamic_features_buf[128];
 	const char *peer_dynamic_features, *chan_dynamic_features;
-	struct ast_flags features;
+	struct ast_flags features = { 0, };
 	struct ast_call_feature feature;
 	if (sense == FEATURE_SENSE_CHAN) {
-		/* Coverity - This uninit_use should be ignored since this macro initializes the flags */
 		ast_copy_flags(&features, &(config->features_caller), AST_FLAGS_ALL);
 	}
 	else {
-		/* Coverity - This uninit_use should be ignored since this macro initializes the flags */
 		ast_copy_flags(&features, &(config->features_callee), AST_FLAGS_ALL);
 	}
 

-- 
To view, visit https://gerrit.asterisk.org/1763
To unsubscribe, visit https://gerrit.asterisk.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: I05162699e1b36bce4145f58a63b47ad19c6975ac
Gerrit-PatchSet: 2
Gerrit-Project: asterisk
Gerrit-Branch: 11
Gerrit-Owner: Joshua Colp <jcolp at digium.com>
Gerrit-Reviewer: Anonymous Coward #1000019
Gerrit-Reviewer: Joshua Colp <jcolp at digium.com>
Gerrit-Reviewer: Matt Jordan <mjordan at digium.com>
Gerrit-Reviewer: Richard Mudgett <rmudgett at digium.com>
Gerrit-Reviewer: Walter Doekes <walter+asterisk at wjd.nu>



More information about the asterisk-commits mailing list