On 01/06/2013 10:39 AM, Mr Dash Four wrote:
> 
>>> 1. ADD(setname:flags) (same with DEL) does not work with sets
>>> containing the "-" character (such sets are accepted by shorewall
>>> anywhere else):
>>>
>>> rules
>>> ~~~~~
>>> ADD(+mickey-mouse:dst,dst) $FW net
>>>
>>> Gives me "ERROR: Expected ipset name (mickey-mouse)".
>>>     
>>
>> Hmmm - That rule compiles error-free for me; git shows that bug was
>> corrected in a commit on October 2 of last year.
>>   
> The patch I am attaching is how I fixed this particular issue when
> compiling shorewall.
> 
>> Agreed. Change will be in Beta 4.
>>   
> Thanks.

My patch is backward-compatible so existing rules that include '+' will
not be rejected.

> 
>>> The above, though, gives me " ERROR: TARGET must be specified".
>>> "Joining" the lines in IELOG using "\" did not have any effect (still
>>> gives me an error).
>>>     
>>
>> If you code action.IELOG as follows, it works:
>>   
> Damn, I tried every other conceivable (random) combination. Will test
> this later tonight. Out of interest though, if I use the alternative
> syntax in its entirety (with curly braces) would that still work?

It should, yes.

-Tom
-- 
Tom Eastep        \ When I die, I want to go like my Grandfather who
Shoreline,         \ died peacefully in his sleep. Not screaming like
Washington, USA     \ all of the passengers in his car
http://shorewall.net \________________________________________________

Attachment: signature.asc
Description: OpenPGP digital signature

------------------------------------------------------------------------------
Master Visual Studio, SharePoint, SQL, ASP.NET, C# 2012, HTML5, CSS,
MVC, Windows 8 Apps, JavaScript and much more. Keep your skills current
with LearnDevNow - 3,200 step-by-step video tutorials by Microsoft
MVPs and experts. ON SALE this month only -- learn more at:
http://p.sf.net/sfu/learnmore_123012
_______________________________________________
Shorewall-devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/shorewall-devel

Reply via email to