2
0
Эх сурвалжийг харах

Revert "core: Initial revision of branch_failure_route"

Remove new branch_failure_route in favour of an event_route

This reverts commit a5946574cb9917f0a9a90ea547c9357f3f6477bd.
Hugh Waite 12 жил өмнө
parent
commit
054af08348
6 өөрчлөгдсөн 3 нэмэгдсэн , 37 устгасан
  1. 0 3
      cfg.lex
  2. 0 24
      cfg.y
  3. 0 4
      route.c
  4. 0 2
      route.h
  5. 1 1
      script_cb.c
  6. 2 3
      script_cb.h

+ 0 - 3
cfg.lex

@@ -200,7 +200,6 @@ ERROR	error
 ROUTE	route
 ROUTE	route
 ROUTE_REQUEST request_route
 ROUTE_REQUEST request_route
 ROUTE_FAILURE failure_route
 ROUTE_FAILURE failure_route
-ROUTE_BRANCH_FAILURE branch_failure_route
 ROUTE_REPLY reply_route
 ROUTE_REPLY reply_route
 ROUTE_ONREPLY onreply_route
 ROUTE_ONREPLY onreply_route
 ROUTE_BRANCH branch_route
 ROUTE_BRANCH branch_route
@@ -624,8 +623,6 @@ IMPORTFILE      "import_file"
 <INITIAL>{ROUTE_REPLY}	{ count(); yylval.strval=yytext; return ROUTE_REPLY; }
 <INITIAL>{ROUTE_REPLY}	{ count(); yylval.strval=yytext; return ROUTE_REPLY; }
 <INITIAL>{ROUTE_FAILURE}	{ count(); yylval.strval=yytext;
 <INITIAL>{ROUTE_FAILURE}	{ count(); yylval.strval=yytext;
 								return ROUTE_FAILURE; }
 								return ROUTE_FAILURE; }
-<INITIAL>{ROUTE_BRANCH_FAILURE}	{ count(); yylval.strval=yytext;
-								return ROUTE_BRANCH_FAILURE; }
 <INITIAL>{ROUTE_BRANCH} { count(); yylval.strval=yytext; return ROUTE_BRANCH; }
 <INITIAL>{ROUTE_BRANCH} { count(); yylval.strval=yytext; return ROUTE_BRANCH; }
 <INITIAL>{ROUTE_SEND} { count(); yylval.strval=yytext; return ROUTE_SEND; }
 <INITIAL>{ROUTE_SEND} { count(); yylval.strval=yytext; return ROUTE_SEND; }
 <INITIAL>{ROUTE_EVENT} { count(); yylval.strval=yytext; return ROUTE_EVENT; }
 <INITIAL>{ROUTE_EVENT} { count(); yylval.strval=yytext; return ROUTE_EVENT; }

+ 0 - 24
cfg.y

@@ -314,7 +314,6 @@ extern char *finame;
 %token ROUTE
 %token ROUTE
 %token ROUTE_REQUEST
 %token ROUTE_REQUEST
 %token ROUTE_FAILURE
 %token ROUTE_FAILURE
-%token ROUTE_BRANCH_FAILURE
 %token ROUTE_ONREPLY
 %token ROUTE_ONREPLY
 %token ROUTE_REPLY
 %token ROUTE_REPLY
 %token ROUTE_BRANCH
 %token ROUTE_BRANCH
@@ -706,7 +705,6 @@ statement:
 	| module_stm
 	| module_stm
 	| {rt=REQUEST_ROUTE;} route_stm
 	| {rt=REQUEST_ROUTE;} route_stm
 	| {rt=FAILURE_ROUTE;} failure_route_stm
 	| {rt=FAILURE_ROUTE;} failure_route_stm
-	| {rt=BRANCH_FAILURE_ROUTE;} branch_failure_route_stm
 	| onreply_route_stm
 	| onreply_route_stm
 	| {rt=BRANCH_ROUTE;} branch_route_stm
 	| {rt=BRANCH_ROUTE;} branch_route_stm
 	| {rt=ONSEND_ROUTE;}   send_route_stm
 	| {rt=ONSEND_ROUTE;}   send_route_stm
@@ -1972,28 +1970,6 @@ failure_route_stm:
 	| ROUTE_FAILURE error { yyerror("invalid failure_route statement"); }
 	| ROUTE_FAILURE error { yyerror("invalid failure_route statement"); }
 	;
 	;
 
 
-branch_failure_route_stm:
-	ROUTE_BRANCH_FAILURE LBRACK route_name RBRACK LBRACE actions RBRACE {
-	#ifdef SHM_MEM
-		if (!shm_initialized() && init_shm()<0) {
-			yyerror("Can't initialize shared memory");
-			YYABORT;
-		}
-	#endif /* SHM_MEM */
-		i_tmp=route_get(&branch_failure_rt, $3);
-		if (i_tmp==-1){
-			yyerror("internal error");
-			YYABORT;
-		}
-		if (branch_failure_rt.rlist[i_tmp]){
-			yyerror("duplicate route");
-			YYABORT;
-		}
-		push($6, &branch_failure_rt.rlist[i_tmp]);
-	}
-	| ROUTE_BRANCH_FAILURE error { yyerror("invalid branch_failure_route statement"); }
-	;
-
 
 
 route_reply_main:	ROUTE_ONREPLY { ; }
 route_reply_main:	ROUTE_ONREPLY { ; }
 		  | ROUTE_REPLY { ; }
 		  | ROUTE_REPLY { ; }

+ 0 - 4
route.c

@@ -102,7 +102,6 @@
 struct route_list main_rt;
 struct route_list main_rt;
 struct route_list onreply_rt;
 struct route_list onreply_rt;
 struct route_list failure_rt;
 struct route_list failure_rt;
-struct route_list branch_failure_rt;
 struct route_list branch_rt;
 struct route_list branch_rt;
 struct route_list onsend_rt;
 struct route_list onsend_rt;
 struct route_list event_rt;
 struct route_list event_rt;
@@ -143,7 +142,6 @@ void destroy_routes()
 	destroy_rlist(&main_rt);
 	destroy_rlist(&main_rt);
 	destroy_rlist(&onreply_rt);
 	destroy_rlist(&onreply_rt);
 	destroy_rlist(&failure_rt);
 	destroy_rlist(&failure_rt);
-	destroy_rlist(&branch_failure_rt);
 	destroy_rlist(&branch_rt);
 	destroy_rlist(&branch_rt);
 	destroy_rlist(&event_rt);
 	destroy_rlist(&event_rt);
 }
 }
@@ -212,8 +210,6 @@ int init_routes()
 		goto error;
 		goto error;
 	if (init_rlist("failure", &failure_rt, FAILURE_RT_NO, RT_HASH_SIZE)<0)
 	if (init_rlist("failure", &failure_rt, FAILURE_RT_NO, RT_HASH_SIZE)<0)
 		goto error;
 		goto error;
-	if (init_rlist("branch_failure", &branch_failure_rt, FAILURE_RT_NO, RT_HASH_SIZE)<0)
-		goto error;
 	if (init_rlist("branch", &branch_rt, BRANCH_RT_NO, RT_HASH_SIZE)<0)
 	if (init_rlist("branch", &branch_rt, BRANCH_RT_NO, RT_HASH_SIZE)<0)
 		goto error;
 		goto error;
 	if (init_rlist("on_send", &onsend_rt, ONSEND_RT_NO, RT_HASH_SIZE)<0)
 	if (init_rlist("on_send", &onsend_rt, ONSEND_RT_NO, RT_HASH_SIZE)<0)

+ 0 - 2
route.h

@@ -53,7 +53,6 @@
 #define ERROR_ROUTE   (1 << 5)
 #define ERROR_ROUTE   (1 << 5)
 #define LOCAL_ROUTE   (1 << 6)
 #define LOCAL_ROUTE   (1 << 6)
 #define CORE_ONREPLY_ROUTE (1 << 7)
 #define CORE_ONREPLY_ROUTE (1 << 7)
-#define BRANCH_FAILURE_ROUTE (1 << 8)
 #define ONREPLY_ROUTE (TM_ONREPLY_ROUTE|CORE_ONREPLY_ROUTE)
 #define ONREPLY_ROUTE (TM_ONREPLY_ROUTE|CORE_ONREPLY_ROUTE)
 #define EVENT_ROUTE   REQUEST_ROUTE
 #define EVENT_ROUTE   REQUEST_ROUTE
 #define ANY_ROUTE     (0xFFFFFFFF)
 #define ANY_ROUTE     (0xFFFFFFFF)
@@ -86,7 +85,6 @@ extern struct route_list main_rt;
 /* main reply route table */
 /* main reply route table */
 extern struct route_list onreply_rt;
 extern struct route_list onreply_rt;
 extern struct route_list failure_rt;
 extern struct route_list failure_rt;
-extern struct route_list branch_failure_rt;
 extern struct route_list branch_rt;
 extern struct route_list branch_rt;
 extern struct route_list onsend_rt;
 extern struct route_list onsend_rt;
 extern struct route_list event_rt;
 extern struct route_list event_rt;

+ 1 - 1
script_cb.c

@@ -53,7 +53,7 @@
 #include "mem/mem.h"
 #include "mem/mem.h"
 
 
 /* Number of cb types = last cb type */
 /* Number of cb types = last cb type */
-#define SCRIPT_CB_NUM	(CB_TYPE_MAX-1)
+#define SCRIPT_CB_NUM	EVENT_CB_TYPE
 
 
 static struct script_cb *pre_script_cb[SCRIPT_CB_NUM];
 static struct script_cb *pre_script_cb[SCRIPT_CB_NUM];
 static struct script_cb *post_script_cb[SCRIPT_CB_NUM];
 static struct script_cb *post_script_cb[SCRIPT_CB_NUM];

+ 2 - 3
script_cb.h

@@ -49,15 +49,14 @@ typedef int (cb_function)(struct sip_msg *msg, unsigned int flags, void *param);
  */
  */
 enum script_cb_flag { REQUEST_CB=1, FAILURE_CB=2, ONREPLY_CB=4,
 enum script_cb_flag { REQUEST_CB=1, FAILURE_CB=2, ONREPLY_CB=4,
 			BRANCH_CB=8, ONSEND_CB=16, ERROR_CB=32,
 			BRANCH_CB=8, ONSEND_CB=16, ERROR_CB=32,
-			LOCAL_CB=64, EVENT_CB=128, BRANCH_FAILURE_CB=256 };
+			LOCAL_CB=64, EVENT_CB=128 };
 
 
 /* Callback types used for executing the callbacks.
 /* Callback types used for executing the callbacks.
  * Keep in sync with script_cb_flag!!!
  * Keep in sync with script_cb_flag!!!
  */
  */
 enum script_cb_type { REQUEST_CB_TYPE=1, FAILURE_CB_TYPE, ONREPLY_CB_TYPE,
 enum script_cb_type { REQUEST_CB_TYPE=1, FAILURE_CB_TYPE, ONREPLY_CB_TYPE,
 			BRANCH_CB_TYPE, ONSEND_CB_TYPE, ERROR_CB_TYPE,
 			BRANCH_CB_TYPE, ONSEND_CB_TYPE, ERROR_CB_TYPE,
-			LOCAL_CB_TYPE, EVENT_CB_TYPE, BRANCH_FAILURE_CB_TYPE,
-			CB_TYPE_MAX};
+			LOCAL_CB_TYPE, EVENT_CB_TYPE };
 
 
 struct script_cb{
 struct script_cb{
 	cb_function *cbf;
 	cb_function *cbf;