[asterisk-commits] russell: branch group/security_events r201184 - in /team/group/security_event...

SVN commits to the Asterisk project asterisk-commits at lists.digium.com
Tue Jun 16 16:34:02 CDT 2009


Author: russell
Date: Tue Jun 16 16:33:55 2009
New Revision: 201184

URL: http://svn.asterisk.org/svn-view/asterisk?view=rev&rev=201184
Log:
Move module field into the common struct

Modified:
    team/group/security_events/channels/chan_sip.c
    team/group/security_events/include/asterisk/security_events_defs.h
    team/group/security_events/main/security_events.c
    team/group/security_events/tests/test_security_events.c

Modified: team/group/security_events/channels/chan_sip.c
URL: http://svn.asterisk.org/svn-view/asterisk/team/group/security_events/channels/chan_sip.c?view=diff&rev=201184&r1=201183&r2=201184
==============================================================================
--- team/group/security_events/channels/chan_sip.c (original)
+++ team/group/security_events/channels/chan_sip.c Tue Jun 16 16:33:55 2009
@@ -12030,8 +12030,8 @@
 		.common.event_type = AST_SECURITY_EVENT_FAILED_ACL,
 		.common.version    = AST_SECURITY_EVENT_FAILED_ACL_VERSION,
 		.common.service    = "SIP",
-
-		.module     = AST_MODULE,
+		.common.module     = AST_MODULE,
+
 		.account_id = peer->name,
 		.session_id = pvt->callid,
 		.acl_name   = acl_name,
@@ -12057,8 +12057,8 @@
 		.common.event_type = AST_SECURITY_EVENT_INVAL_ACCT_ID,
 		.common.version    = AST_SECURITY_EVENT_INVAL_ACCT_ID_VERSION,
 		.common.service    = "SIP",
-
-		.module     = AST_MODULE,
+		.common.module     = AST_MODULE,
+
 		.account_id = peer_name,
 		.session_id = pvt->callid,
 		.session_tv = &pvt->session_tv,

Modified: team/group/security_events/include/asterisk/security_events_defs.h
URL: http://svn.asterisk.org/svn-view/asterisk/team/group/security_events/include/asterisk/security_events_defs.h?view=diff&rev=201184&r1=201183&r2=201184
==============================================================================
--- team/group/security_events/include/asterisk/security_events_defs.h (original)
+++ team/group/security_events/include/asterisk/security_events_defs.h Tue Jun 16 16:33:55 2009
@@ -163,10 +163,16 @@
 	uint32_t version;
 	/*!
 	 * \brief Service that generated the event
+	 * \note Required
 	 *
-	 * Examples: "SIP"
+	 * Examples: "SIP", "AMI"
 	 */
 	const char *service;
+	/*!
+	 * \brief Module, Normally the AST_MODULE define
+	 * \note optional
+	 */
+	const char *module;
 };
 
 /*!
@@ -181,11 +187,6 @@
 	/*! \brief Common security event descriptor elements */
 	struct ast_security_event_common common;
 	/*!
-	 * \brief Module, Normally the AST_MODULE define
-	 * \note optional
-	 */
-	const char *module;
-	/*!
 	 * \brief Account ID, specific to the service type
 	 * \note required
 	 */
@@ -229,11 +230,6 @@
 	/*! \brief Common security event descriptor elements */
 	struct ast_security_event_common common;
 	/*!
-	 * \brief Module, Normally the AST_MODULE define
-	 * \note optional
-	 */
-	const char *module;
-	/*!
 	 * \brief Account ID, specific to the service type
 	 * \note required
 	 */
@@ -272,11 +268,6 @@
 	/*! \brief Common security event descriptor elements */
 	struct ast_security_event_common common;
 	/*!
-	 * \brief Module, Normally the AST_MODULE define
-	 * \note optional
-	 */
-	const char *module;
-	/*!
 	 * \brief Account ID, specific to the service type
 	 * \note required
 	 */
@@ -315,11 +306,6 @@
 	/*! \brief Common security event descriptor elements */
 	struct ast_security_event_common common;
 	/*!
-	 * \brief Module, Normally the AST_MODULE define
-	 * \note optional
-	 */
-	const char *module;
-	/*!
 	 * \brief Account ID, specific to the service type
 	 * \note required
 	 */
@@ -358,11 +344,6 @@
 	/*! \brief Common security event descriptor elements */
 	struct ast_security_event_common common;
 	/*!
-	 * \brief Module, Normally the AST_MODULE define
-	 * \note optional
-	 */
-	const char *module;
-	/*!
 	 * \brief Account ID, specific to the service type
 	 * \note required
 	 */
@@ -401,11 +382,6 @@
 	/*! \brief Common security event descriptor elements */
 	struct ast_security_event_common common;
 	/*!
-	 * \brief Module, Normally the AST_MODULE define
-	 * \note optional
-	 */
-	const char *module;
-	/*!
 	 * \brief Account ID, specific to the service type
 	 * \note required
 	 */
@@ -449,11 +425,6 @@
 	/*! \brief Common security event descriptor elements */
 	struct ast_security_event_common common;
 	/*!
-	 * \brief Module, Normally the AST_MODULE define
-	 * \note optional
-	 */
-	const char *module;
-	/*!
 	 * \brief Account ID, specific to the service type
 	 * \note required
 	 */
@@ -502,11 +473,6 @@
 	/*! \brief Common security event descriptor elements */
 	struct ast_security_event_common common;
 	/*!
-	 * \brief Module, Normally the AST_MODULE define
-	 * \note optional
-	 */
-	const char *module;
-	/*!
 	 * \brief Account ID, specific to the service type
 	 * \note required
 	 */
@@ -550,11 +516,6 @@
 	/*! \brief Common security event descriptor elements */
 	struct ast_security_event_common common;
 	/*!
-	 * \brief Module, Normally the AST_MODULE define
-	 * \note optional
-	 */
-	const char *module;
-	/*!
 	 * \brief Account ID, specific to the service type
 	 * \note optional
 	 */
@@ -603,11 +564,6 @@
 	/*! \brief Common security event descriptor elements */
 	struct ast_security_event_common common;
 	/*!
-	 * \brief Module, Normally the AST_MODULE define
-	 * \note optional
-	 */
-	const char *module;
-	/*!
 	 * \brief Account ID, specific to the service type
 	 * \note required
 	 */
@@ -646,11 +602,6 @@
 	/*! \brief Common security event descriptor elements */
 	struct ast_security_event_common common;
 	/*!
-	 * \brief Module, Normally the AST_MODULE define
-	 * \note optional
-	 */
-	const char *module;
-	/*!
 	 * \brief Account ID, specific to the service type
 	 * \note required
 	 */
@@ -694,11 +645,6 @@
 	/*! \brief Common security event descriptor elements */
 	struct ast_security_event_common common;
 	/*!
-	 * \brief Module, Normally the AST_MODULE define
-	 * \note optional
-	 */
-	const char *module;
-	/*!
 	 * \brief Account ID, specific to the service type
 	 * \note required
 	 */
@@ -751,11 +697,6 @@
 	#define AST_SECURITY_EVENT_INVAL_PASSWORD_VERSION 1
 	/*! \brief Common security event descriptor elements */
 	struct ast_security_event_common common;
-	/*!
-	 * \brief Module, Normally the AST_MODULE define
-	 * \note optional
-	 */
-	const char *module;
 	/*!
 	 * \brief Account ID, specific to the service type
 	 * \note required

Modified: team/group/security_events/main/security_events.c
URL: http://svn.asterisk.org/svn-view/asterisk/team/group/security_events/main/security_events.c?view=diff&rev=201184&r1=201183&r2=201184
==============================================================================
--- team/group/security_events/main/security_events.c (original)
+++ team/group/security_events/main/security_events.c Tue Jun 16 16:33:55 2009
@@ -63,7 +63,7 @@
 		{ AST_EVENT_IE_END, 0 }
 	},
 	.optional_ies = {
-		{ AST_EVENT_IE_MODULE, SEC_EVT_FIELD(failed_acl, module) },
+		{ AST_EVENT_IE_MODULE, SEC_EVT_FIELD(common, module) },
 		{ AST_EVENT_IE_ACL_NAME, SEC_EVT_FIELD(failed_acl, acl_name) },
 		{ AST_EVENT_IE_SESSION_TV, SEC_EVT_FIELD(failed_acl, session_tv) },
 		{ AST_EVENT_IE_END, 0 }
@@ -86,7 +86,7 @@
 		{ AST_EVENT_IE_END, 0 }
 	},
 	.optional_ies = {
-		{ AST_EVENT_IE_MODULE, SEC_EVT_FIELD(inval_acct_id, module) },
+		{ AST_EVENT_IE_MODULE, SEC_EVT_FIELD(common, module) },
 		{ AST_EVENT_IE_SESSION_TV, SEC_EVT_FIELD(inval_acct_id, session_tv) },
 		{ AST_EVENT_IE_END, 0 }
 	},
@@ -108,7 +108,7 @@
 		{ AST_EVENT_IE_END, 0 }
 	},
 	.optional_ies = {
-		{ AST_EVENT_IE_MODULE, SEC_EVT_FIELD(session_limit, module) },
+		{ AST_EVENT_IE_MODULE, SEC_EVT_FIELD(common, module) },
 		{ AST_EVENT_IE_SESSION_TV, SEC_EVT_FIELD(session_limit, session_tv) },
 		{ AST_EVENT_IE_END, 0 }
 	},
@@ -130,7 +130,7 @@
 		{ AST_EVENT_IE_END, 0 }
 	},
 	.optional_ies = {
-		{ AST_EVENT_IE_MODULE, SEC_EVT_FIELD(mem_limit, module) },
+		{ AST_EVENT_IE_MODULE, SEC_EVT_FIELD(common, module) },
 		{ AST_EVENT_IE_SESSION_TV, SEC_EVT_FIELD(mem_limit, session_tv) },
 		{ AST_EVENT_IE_END, 0 }
 	},
@@ -152,7 +152,7 @@
 		{ AST_EVENT_IE_END, 0 }
 	},
 	.optional_ies = {
-		{ AST_EVENT_IE_MODULE, SEC_EVT_FIELD(load_avg, module) },
+		{ AST_EVENT_IE_MODULE, SEC_EVT_FIELD(common, module) },
 		{ AST_EVENT_IE_SESSION_TV, SEC_EVT_FIELD(load_avg, session_tv) },
 		{ AST_EVENT_IE_END, 0 }
 	},
@@ -175,7 +175,7 @@
 		{ AST_EVENT_IE_END, 0 }
 	},
 	.optional_ies = {
-		{ AST_EVENT_IE_MODULE, SEC_EVT_FIELD(req_no_support, module) },
+		{ AST_EVENT_IE_MODULE, SEC_EVT_FIELD(common, module) },
 		{ AST_EVENT_IE_SESSION_TV, SEC_EVT_FIELD(req_no_support, session_tv) },
 		{ AST_EVENT_IE_END, 0 }
 	},
@@ -198,7 +198,7 @@
 		{ AST_EVENT_IE_END, 0 }
 	},
 	.optional_ies = {
-		{ AST_EVENT_IE_MODULE, SEC_EVT_FIELD(req_not_allowed, module) },
+		{ AST_EVENT_IE_MODULE, SEC_EVT_FIELD(common, module) },
 		{ AST_EVENT_IE_SESSION_TV, SEC_EVT_FIELD(req_not_allowed, session_tv) },
 		{ AST_EVENT_IE_REQUEST_PARAMS, SEC_EVT_FIELD(req_not_allowed, request_params) },
 		{ AST_EVENT_IE_END, 0 }
@@ -222,7 +222,7 @@
 		{ AST_EVENT_IE_END, 0 }
 	},
 	.optional_ies = {
-		{ AST_EVENT_IE_MODULE, SEC_EVT_FIELD(auth_method_not_allowed, module) },
+		{ AST_EVENT_IE_MODULE, SEC_EVT_FIELD(common, module) },
 		{ AST_EVENT_IE_SESSION_TV, SEC_EVT_FIELD(auth_method_not_allowed, session_tv) },
 		{ AST_EVENT_IE_END, 0 }
 	},
@@ -244,7 +244,7 @@
 		{ AST_EVENT_IE_END, 0 }
 	},
 	.optional_ies = {
-		{ AST_EVENT_IE_MODULE, SEC_EVT_FIELD(req_bad_format, module) },
+		{ AST_EVENT_IE_MODULE, SEC_EVT_FIELD(common, module) },
 		{ AST_EVENT_IE_SESSION_TV, SEC_EVT_FIELD(req_bad_format, session_tv) },
 		{ AST_EVENT_IE_ACCOUNT_ID, SEC_EVT_FIELD(req_bad_format, account_id) },
 		{ AST_EVENT_IE_REQUEST_PARAMS, SEC_EVT_FIELD(req_bad_format, request_params) },
@@ -268,7 +268,7 @@
 		{ AST_EVENT_IE_END, 0 }
 	},
 	.optional_ies = {
-		{ AST_EVENT_IE_MODULE, SEC_EVT_FIELD(successful_auth, module) },
+		{ AST_EVENT_IE_MODULE, SEC_EVT_FIELD(common, module) },
 		{ AST_EVENT_IE_SESSION_TV, SEC_EVT_FIELD(successful_auth, session_tv) },
 		{ AST_EVENT_IE_END, 0 }
 	},
@@ -291,7 +291,7 @@
 		{ AST_EVENT_IE_END, 0 }
 	},
 	.optional_ies = {
-		{ AST_EVENT_IE_MODULE, SEC_EVT_FIELD(unexpected_addr, module) },
+		{ AST_EVENT_IE_MODULE, SEC_EVT_FIELD(common, module) },
 		{ AST_EVENT_IE_SESSION_TV, SEC_EVT_FIELD(unexpected_addr, session_tv) },
 		{ AST_EVENT_IE_END, 0 }
 	},
@@ -316,7 +316,7 @@
 		{ AST_EVENT_IE_END, 0 }
 	},
 	.optional_ies = {
-		{ AST_EVENT_IE_MODULE, SEC_EVT_FIELD(chal_resp_failed, module) },
+		{ AST_EVENT_IE_MODULE, SEC_EVT_FIELD(common, module) },
 		{ AST_EVENT_IE_SESSION_TV, SEC_EVT_FIELD(chal_resp_failed, session_tv) },
 		{ AST_EVENT_IE_END, 0 }
 	},
@@ -338,7 +338,7 @@
 		{ AST_EVENT_IE_END, 0 }
 	},
 	.optional_ies = {
-		{ AST_EVENT_IE_MODULE, SEC_EVT_FIELD(inval_password, module) },
+		{ AST_EVENT_IE_MODULE, SEC_EVT_FIELD(common, module) },
 		{ AST_EVENT_IE_SESSION_TV, SEC_EVT_FIELD(inval_password, session_tv) },
 		{ AST_EVENT_IE_END, 0 }
 	},

Modified: team/group/security_events/tests/test_security_events.c
URL: http://svn.asterisk.org/svn-view/asterisk/team/group/security_events/tests/test_security_events.c?view=diff&rev=201184&r1=201183&r2=201184
==============================================================================
--- team/group/security_events/tests/test_security_events.c (original)
+++ team/group/security_events/tests/test_security_events.c Tue Jun 16 16:33:55 2009
@@ -80,8 +80,8 @@
 		.common.event_type = AST_SECURITY_EVENT_FAILED_ACL,
 		.common.version    = AST_SECURITY_EVENT_FAILED_ACL_VERSION,
 		.common.service    = "TEST",
-
-		.module     = AST_MODULE,
+		.common.module     = AST_MODULE,
+
 		.account_id = "Username",
 		.session_id = "Session123",
 		.acl_name   = "TEST_ACL",
@@ -118,8 +118,8 @@
 		.common.event_type = AST_SECURITY_EVENT_INVAL_ACCT_ID,
 		.common.version    = AST_SECURITY_EVENT_INVAL_ACCT_ID_VERSION,
 		.common.service    = "TEST",
-
-		.module     = AST_MODULE,
+		.common.module     = AST_MODULE,
+
 		.account_id = "FakeUser",
 		.session_id = "Session456",
 		.session_tv = &session_tv,
@@ -155,8 +155,8 @@
 		.common.event_type = AST_SECURITY_EVENT_SESSION_LIMIT,
 		.common.version    = AST_SECURITY_EVENT_SESSION_LIMIT_VERSION,
 		.common.service    = "TEST",
-
-		.module     = AST_MODULE,
+		.common.module     = AST_MODULE,
+
 		.account_id = "Jenny",
 		.session_id = "8675309",
 		.session_tv = &session_tv,
@@ -192,8 +192,8 @@
 		.common.event_type = AST_SECURITY_EVENT_MEM_LIMIT,
 		.common.version    = AST_SECURITY_EVENT_MEM_LIMIT_VERSION,
 		.common.service    = "TEST",
-
-		.module     = AST_MODULE,
+		.common.module     = AST_MODULE,
+
 		.account_id = "Felix",
 		.session_id = "Session2604",
 		.session_tv = &session_tv,
@@ -229,8 +229,8 @@
 		.common.event_type = AST_SECURITY_EVENT_LOAD_AVG,
 		.common.version    = AST_SECURITY_EVENT_LOAD_AVG_VERSION,
 		.common.service    = "TEST",
-
-		.module     = AST_MODULE,
+		.common.module     = AST_MODULE,
+
 		.account_id = "GuestAccount",
 		.session_id = "XYZ123",
 		.session_tv = &session_tv,
@@ -266,8 +266,8 @@
 		.common.event_type = AST_SECURITY_EVENT_REQ_NO_SUPPORT,
 		.common.version    = AST_SECURITY_EVENT_REQ_NO_SUPPORT_VERSION,
 		.common.service    = "TEST",
-
-		.module     = AST_MODULE,
+		.common.module     = AST_MODULE,
+
 		.account_id = "George",
 		.session_id = "alksdjf023423h4lka0df",
 		.session_tv = &session_tv,
@@ -304,8 +304,8 @@
 		.common.event_type = AST_SECURITY_EVENT_REQ_NOT_ALLOWED,
 		.common.version    = AST_SECURITY_EVENT_REQ_NOT_ALLOWED_VERSION,
 		.common.service    = "TEST",
-
-		.module     = AST_MODULE,
+		.common.module     = AST_MODULE,
+
 		.account_id = "George",
 		.session_id = "alksdjf023423h4lka0df",
 		.session_tv = &session_tv,
@@ -343,8 +343,8 @@
 		.common.event_type = AST_SECURITY_EVENT_AUTH_METHOD_NOT_ALLOWED,
 		.common.version    = AST_SECURITY_EVENT_AUTH_METHOD_NOT_ALLOWED_VERSION,
 		.common.service    = "TEST",
-
-		.module     = AST_MODULE,
+		.common.module     = AST_MODULE,
+
 		.account_id = "Bob",
 		.session_id = "010101010101",
 		.session_tv = &session_tv,
@@ -381,8 +381,8 @@
 		.common.event_type = AST_SECURITY_EVENT_REQ_BAD_FORMAT,
 		.common.version    = AST_SECURITY_EVENT_REQ_BAD_FORMAT_VERSION,
 		.common.service    = "TEST",
-
-		.module     = AST_MODULE,
+		.common.module     = AST_MODULE,
+
 		.account_id = "Larry",
 		.session_id = "838383fhfhf83hf8h3f8h",
 		.session_tv = &session_tv,
@@ -420,8 +420,8 @@
 		.common.event_type = AST_SECURITY_EVENT_SUCCESSFUL_AUTH,
 		.common.version    = AST_SECURITY_EVENT_SUCCESSFUL_AUTH_VERSION,
 		.common.service    = "TEST",
-
-		.module     = AST_MODULE,
+		.common.module     = AST_MODULE,
+
 		.account_id = "ValidUser",
 		.session_id = "Session456",
 		.session_tv = &session_tv,
@@ -460,8 +460,8 @@
 		.common.event_type = AST_SECURITY_EVENT_UNEXPECTED_ADDR,
 		.common.version    = AST_SECURITY_EVENT_UNEXPECTED_ADDR_VERSION,
 		.common.service    = "TEST",
-
-		.module     = AST_MODULE,
+		.common.module     = AST_MODULE,
+
 		.account_id = "CoolUser",
 		.session_id = "Session789",
 		.session_tv = &session_tv,
@@ -504,8 +504,8 @@
 		.common.event_type = AST_SECURITY_EVENT_CHAL_RESP_FAILED,
 		.common.version    = AST_SECURITY_EVENT_CHAL_RESP_FAILED_VERSION,
 		.common.service    = "TEST",
-
-		.module     = AST_MODULE,
+		.common.module     = AST_MODULE,
+
 		.account_id = "SuperDuperUser",
 		.session_id = "Session1231231231",
 		.session_tv = &session_tv,
@@ -544,8 +544,8 @@
 		.common.event_type = AST_SECURITY_EVENT_INVAL_PASSWORD,
 		.common.version    = AST_SECURITY_EVENT_INVAL_PASSWORD_VERSION,
 		.common.service    = "TEST",
-
-		.module     = AST_MODULE,
+		.common.module     = AST_MODULE,
+
 		.account_id = "AccountIDGoesHere",
 		.session_id = "SessionIDGoesHere",
 		.session_tv = &session_tv,




More information about the asterisk-commits mailing list