event_routing: Fix missing lock_release() on error case

liviuchircu · liviuchircu · commit 6ea748c768fe · 2021-12-20T13:08:39.000+02:00
Credits to @ryancaicse for the report in #2714
diff --git a/modules/event_routing/ebr_data.c b/modules/event_routing/ebr_data.c
@@ -143,7 +143,7 @@ int init_ebr_event( ebr_event *ev )
 	if (evi_event_subscribe( ev->event_name, sock, 0, 0) < 0) {
 		LM_ERR("cannot subscribe to event %.*s\n",
 			ev->event_name.len, ev->event_name.s);
-		return -1;
+		goto error;
 	}
 
 	lock_release( &(ev->lock) );

Original file line number	Diff line number	Diff line change
`@@ -143,7 +143,7 @@ int init_ebr_event( ebr_event *ev )`
`143`	`143`	`if (evi_event_subscribe( ev->event_name, sock, 0, 0) < 0) {`
`144`	`144`	`LM_ERR("cannot subscribe to event %.*s\n",`
`145`	`145`	`ev->event_name.len, ev->event_name.s);`
`146`		`- return -1;`
	`146`	`+ goto error;`
`147`	`147`	`}`
`148`	`148`
`149`	`149`	`lock_release( &(ev->lock) );`