openggsn[master]: Signal handler for SIGTERM, SIGHUP and SIGINT added to SGSNE...

Jovan Bunjevacki gerrit-no-reply at lists.osmocom.org
Fri Feb 24 19:13:48 UTC 2017


Patch Set 1:

Hello Harald.

Thanks for the comment.
In function "create_pdp_conf" when everything is set up there's this line: 	state = 2;	/* Connected */
Signal handler changes the value of variable "state" to 3 if it is equal to 2.
This has the same effect as if run time of sgsnemu set in options has elapsed.

Sorry for "cosmetic" errors, I'm not so familiar with code style you're using.
I will correct this.



void signal_handler(int signo)
{
	if(state == 2) state = 3;  /* Tell main loop to finish. */ }


/* Send off disconnect */
if (3 == state) {
	state = 4;
	stoptime = time(NULL) + 5;	/* Extra seconds to allow disconnect */
	for (n = 0; n < options.contexts; n++) {
		/* Delete context */
		printf("Disconnecting PDP context #%d\n", n);
		gtp_delete_context_req(gsn, iparr[n].pdp, NULL,
				       1);
		if ((options.pinghost.s_addr != 0)
		    && ntransmitted)
			ping_finish();
	}
}

-- 
To view, visit https://gerrit.osmocom.org/1902
To unsubscribe, visit https://gerrit.osmocom.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I546dc0a8d814527b9a96943a52f2f6b700285167
Gerrit-PatchSet: 1
Gerrit-Project: openggsn
Gerrit-Branch: master
Gerrit-Owner: Jovan Bunjevacki <j.bunjevacki at vipmobile.rs>
Gerrit-Reviewer: Harald Welte <laforge at gnumonks.org>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: Jovan Bunjevacki <j.bunjevacki at vipmobile.rs>
Gerrit-HasComments: No


More information about the gerrit-log mailing list