[sr-dev] [tracker] Comment added: Nathelper: modparam natping_processes max value

[sip-router]

THIS IS AN AUTOMATED MESSAGE, DO NOT REPLY.

The following task has a new comment added:

FS#321 - Nathelper: modparam natping_processes max value
User who did this - Víctor Seva (linuxmaniac)

----------
from modules/nathelper.c:

if (natping_processes>=8) {
  LM_ERR("too many natping processes (%d) max=8\n", natping_processes);
  return -1;
}

and at mod_init:

	if (rank==PROC_MAIN && natping_interval > 0) {
		for( i=0 ; i<natping_processes ; i++ ) {
			if(fork_dummy_timer(PROC_TIMER, "TIMER NH", 1 /*socks flag*/,
								nh_timer, (void*)(unsigned long)i,
								1 /*sec*/)<0) {
				LM_ERR("failed to register timer routine as process\n");
				return -1;
				/* error */
			}
		}
	}

So... I think this is just what we need:

diff --git a/modules/nathelper/nathelper.c b/modules/nathelper/nathelper.c
index cff54d8..948c9c0 100644
--- a/modules/nathelper/nathelper.c
+++ b/modules/nathelper/nathelper.c
@@ -671,7 +671,7 @@ mod_init(void)
                                " set in usrloc module\n");
                        return -1;
                }
-               if (natping_processes>=8) {
+               if (natping_processes>8) {
                        LM_ERR("too many natping processes (%d) max=8\n",
                                natping_processes);
                        return -1;

----------

More information can be found at the following URL:
http://sip-router.org/tracker/index.php?do=details&task_id=321#comment974

You are receiving this message because you have requested it from the Flyspray bugtracking system.  If you did not expect this message or don't want to receive mails in future, you can change your notification settings at the URL shown above.