Patch for individual session requests from chilli.sock for consideration.

Scott Tester scott at smbc.com.au
Thu May 21 07:07:57 UTC 2009


I am working at integrating coovachilli with dansguardian.  Sending the 
list command to the socket returns the data for all active sessions.  As 
I want to keep the socket communication to a minimum, I've modified 
coovachilli (see attached patch) to return only the ip or mac requested.
Would you consider merging these changes* for two reasons:
1. Other's can benifit.
2. I don't have to modify the source each time I upgrade coova on my system.


NOTES:
1.  There are two lines that didn't want to compile due to a required 
third argument when open() uses O_CREAT, (chilli.patch:9 & 
chilli.patch:18).  This is not the main reason for this patch, but I've 
included these fixes anyway.
2.  I'm not fussed if anyone has a better idea for command names instead 
of 'listip' and 'listmac'.  Same goes for CMDSOCK_ENTRY_FOR_MAC and 
CMDSOCK_ENTRY_FOR_IP.


* I understand that you will want to review the code.  In fact, that 
would be appreciated - if you don't mind.
-- 

Scott Tester

IT Department

SMBC  www.smbc.com.au <http://www.smbc.com.au>

Sydney Missionary and Bible College

43 Badminton Road, Croydon NSW 2132

Phone: 02 9747 4780  ext 147

Mobile: 0411 145 584

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.coova.org/pipermail/chilli/attachments/20090521/df89b00e/attachment.htm>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: chilli.patch
Type: text/x-patch
Size: 4836 bytes
Desc: not available
URL: <http://lists.coova.org/pipermail/chilli/attachments/20090521/df89b00e/attachment.bin>


More information about the Chilli mailing list