[OpenSER-Devel] [ openser-Bugs-1890988 ] [permissions] "default_allow/deny_file" is not a parameter

[SourceForge.net]

Bugs item #1890988, was opened at 2008-02-11 10:57
Message generated for change (Settings changed) made by bogdan_iancu
You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=743020&aid=1890988&group_id=139143

Please note that this message will contain a full copy of the comment thread,
including the initial issue submission, for this request,
not just the latest update.
Category: modules
Group: ver devel
Status: Open
>Resolution: Fixed
Priority: 5
Private: No
Submitted By: Iñaki Baz (ibc_sf)
>Assigned to: Bogdan-Andrei Iancu (bogdan_iancu)
Summary: [permissions] "default_allow/deny_file" is not a parameter

Initial Comment:
There are two parameters in "permissions" module:

- default_allow_file (string)
- default_deny_file (string)

They are supposed to contain the custom path for rule files.

But they are not valid at all since code don't look for them, but for a constant values:


permissions.h:
------------------
#define DEFAULT_ALLOW_FILE "permissions.allow"
#define DEFAULT_DENY_FILE  "permissions.deny"
------------------


permissions.c:
------------------
static char* default_allow_file = DEFAULT_ALLOW_FILE;
static char* default_deny_file = DEFAULT_DENY_FILE;

[...]

static param_export_t params[] = {
	{"default_allow_file", STR_PARAM, &default_allow_file},
	{"default_deny_file",  STR_PARAM, &default_deny_file },

[...]

allow[0].filename = get_pathname(DEFAULT_ALLOW_FILE);
[...]
deny[0].filename = get_pathname(DEFAULT_DENY_FILE);
------------------


In fact, I set those parameter to any value and it's not looked for when starting OpenSer:

---------------
modparam("permissions", "default_allow_file", "/etc/MY_CUSTOM_LOCATION_ALLOW_FILE")
---------------

~$ /etc/init.d/openser start
...
 WARNING:permissions:mod_init: default allow file (/etc/openser/permissions.allow) not found => empty rule set
...

----------------------------------------------------------------------

>Comment By: Bogdan-Andrei Iancu (bogdan_iancu)
Date: 2008-02-11 14:43

Message:
Logged In: YES 
user_id=1275325
Originator: NO

Hi Inaki,

I made a fix on SVN trunk - please test and let me know if ok. If so, I
will do the backports.

Thanks and regards,
Bogdan

----------------------------------------------------------------------

You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=743020&aid=1890988&group_id=139143