Re: [PATCH] show IDS rule names correctly in WebUI log

[Michael Tremer <michael.tremer@ipfire.org>]

[-- Attachment #1: Type: text/plain, Size: 1803 bytes --]

Hi,

the bug ID is good, but it didn't have a precise explanation either.

I just wanted this on the record so that we know what this patch was supposed to
be doing.

Yes, please send again.

Best,
-Michael

On Mon, 2017-11-20 at 19:13 +0100, Peter Müller wrote:
> Hello Michael,
> 
> Emerging Threats changed some snort signature IDs to 5 digits
> a while, which were not displayed in the WebUI since "([0-9]{1,4})"
> did not match on it.
> 
> This patch adjusts the regular expression so they are displayed
> correctly now.
> 
> Should I send in a second version with this description? Thought
> the bug ID would be enough...
> 
> Best regards, 
> Peter Müller
> 
> > Hello,
> > 
> > can you explain what this patch is supposed to change?
> > 
> > -Michael
> > 
> > On Sun, 2017-11-19 at 18:04 +0100, Peter Müller wrote:
> > > Fixes #11519.
> > > 
> > > Signed-off-by: Peter Müller <peter.mueller(a)link38.eu>
> > > ---
> > >  html/cgi-bin/logs.cgi/ids.dat | 2 +-
> > >  1 file changed, 1 insertion(+), 1 deletion(-)
> > > 
> > > diff --git a/html/cgi-bin/logs.cgi/ids.dat b/html/cgi-bin/logs.cgi/ids.dat
> > > index 44b3abdac..98176d690 100644
> > > --- a/html/cgi-bin/logs.cgi/ids.dat
> > > +++ b/html/cgi-bin/logs.cgi/ids.dat
> > > @@ -387,7 +387,7 @@ sub processevent
> > >  				}
> > >  				($title,$classification,$priority,$date,$
> > > time
> > > ,$srcip,$srcport,$destip,$destport, $sid) =
> > > ("n/a","n/a","n/a","n/a","n/a","n/a","n/a","n/a","n/a", "n/a");
> > >  				@refs = ();
> > > -				$_ =~ m/:([0-9]{1,4})\] (.*) \[\*\*\]/;
> > > +				$_ =~ m/:([0-9]{1,5})\] (.*) \[\*\*\]/;
> > >  				$title = &Header::cleanhtml($2,"y");
> > >  			}
> > >  			if ($_ =~ m/Classification: (.*)\] \[Priority:
> > > (\d)\]/) {  
> 
> 

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 833 bytes --]