Skip Menu |

This queue is for tickets about the PIX-Walker CPAN distribution.

Report information
The Basics
Id: 25397
Status: resolved
Priority: 0/
Queue: PIX-Walker
People
Owner: Nobody in particular
Requestors: ryan [...] muppethouse.com
Cc:
AdminCc:
Bug Information
Severity: (no value)
Broken in: (no value)
Fixed in: (no value)

History Show all quoted text
  Mon Mar 12 21:37:18 2007 ryan [...] muppethouse.com - Ticket created
Subject: PIX::Walker - Descriptions & Names
Date: Mon, 12 Mar 2007 21:36:59 -0400
To: bug-PIX-Walker [...] rt.cpan.org
From: "Ryan Shea" <ryan [...] muppethouse.com>
OS Ubuntu Linux Breezy Badger Perl 5.8.8 PIX::Walker from CPAN today FWSM config file version 3.1(4) which I think is roughly equivalent to 7.2. This module assumes that what follows the word host is going to be an IP address, when in fact for those who put in names at the top, it often is not: Use of uninitialized value in left bitshift (<<) at /usr/local/share/perl/5.8.8/PIX/Object/network.pm line 105. Use of uninitialized value in bitwise or (|) at /usr/local/share/perl/5.8.8/PIX/Object/network.pm line 105. Use of uninitialized value in left bitshift (<<) at /usr/local/share/perl/5.8.8/PIX/Object/network.pm line 105. Argument "vs1cricket1" isn't numeric in left bitshift (<<) at /usr/local/share/perl/5.8.8/PIX/Object/network.pm line 105. I would expect that the name to IP mappings should be kept as host objects. Also it looks like the parser cannot handle a description for a service object-group: win-update-ports: Unknown service object line: description Window update ports the object-group looks like this object-group service win-update-ports tcp description Window update ports port-object eq www port-object eq https I wrote a really terrible pix 7 config parser that works and does this stuff and wish I had found this module first. If I get a chance I'll see if I can make a patch for your module to make it work with names and descriptions, but I don't want to get in over my head.
  Fri Nov 07 08:24:12 2008 lifo101 [...] gmail.com - Correspondence added
I can't reproduce the error you show with the bit shifting. The parser does in fact try to de-alias any IP on an acl line and does not just assume a host is an IP. Your second error was fixed in another bug report and is due to the leading space missing in front of 'description' when PIX::Walker tries to parse the config. This is also fixed. PIX::Walker has not been updated on CPAN yet with these changes. On Mon Mar 12 21:37:18 2007, ryan@muppethouse.com wrote: Show quoted text
> OS Ubuntu Linux Breezy Badger > Perl 5.8.8 > PIX::Walker from CPAN today > > FWSM config file version 3.1(4) which I think is roughly equivalent to > 7.2. > > This module assumes that what follows the word host is going to be an > IP address, when in fact for those who put in names at the top, it > often is not: > > Use of uninitialized value in left bitshift (<<) at > /usr/local/share/perl/5.8.8/PIX/Object/network.pm line 105. > Use of uninitialized value in bitwise or (|) at > /usr/local/share/perl/5.8.8/PIX/Object/network.pm line 105. > Use of uninitialized value in left bitshift (<<) at > /usr/local/share/perl/5.8.8/PIX/Object/network.pm line 105. > Argument "vs1cricket1" isn't numeric in left bitshift (<<) at > /usr/local/share/perl/5.8.8/PIX/Object/network.pm line 105. > > I would expect that the name to IP mappings should be kept as host > objects. > > Also it looks like the parser cannot handle a description for a > service object-group: > > win-update-ports: Unknown service object line: description Window > update ports > > the object-group looks like this > > object-group service win-update-ports tcp > description Window update ports > port-object eq www > port-object eq https > > > I wrote a really terrible pix 7 config parser that works and does this > stuff and wish I had found this module first. If I get a chance I'll > see if I can make a patch for your module to make it work with names > and descriptions, but I don't want to get in over my head.
  Fri Nov 07 08:24:14 2008 The RT System itself - Status changed from 'new' to 'open'
  Fri Nov 07 08:24:14 2008 lifo101 [...] gmail.com - Status changed from 'open' to 'rejected'
  Fri Nov 07 08:55:43 2008 ryan [...] muppethouse.com - Correspondence added
Subject: Re: [rt.cpan.org #25397] PIX::Walker - Descriptions & Names
Date: Fri, 7 Nov 2008 08:55:28 -0500
To: bug-PIX-Walker [...] rt.cpan.org
From: "Ryan Shea" <ryan [...] muppethouse.com>
Thank you so much for looking at these. On Fri, Nov 7, 2008 at 8:24 AM, Jason Morriss via RT < bug-PIX-Walker@rt.cpan.org> wrote: Show quoted text
> <URL: http://rt.cpan.org/Ticket/Display.html?id=25397 > > > I can't reproduce the error you show with the bit shifting. The parser > does in fact try to de-alias any IP on an acl line and does not just > assume a host is an IP. > > Your second error was fixed in another bug report and is due to the > leading space missing in front of 'description' when PIX::Walker tries > to parse the config. This is also fixed. > > PIX::Walker has not been updated on CPAN yet with these changes. > > On Mon Mar 12 21:37:18 2007, ryan@muppethouse.com wrote:
> > OS Ubuntu Linux Breezy Badger > > Perl 5.8.8 > > PIX::Walker from CPAN today > > > > FWSM config file version 3.1(4) which I think is roughly equivalent to > > 7.2. > > > > This module assumes that what follows the word host is going to be an > > IP address, when in fact for those who put in names at the top, it > > often is not: > > > > Use of uninitialized value in left bitshift (<<) at > > /usr/local/share/perl/5.8.8/PIX/Object/network.pm line 105. > > Use of uninitialized value in bitwise or (|) at > > /usr/local/share/perl/5.8.8/PIX/Object/network.pm line 105. > > Use of uninitialized value in left bitshift (<<) at > > /usr/local/share/perl/5.8.8/PIX/Object/network.pm line 105. > > Argument "vs1cricket1" isn't numeric in left bitshift (<<) at > > /usr/local/share/perl/5.8.8/PIX/Object/network.pm line 105. > > > > I would expect that the name to IP mappings should be kept as host > > objects. > > > > Also it looks like the parser cannot handle a description for a > > service object-group: > > > > win-update-ports: Unknown service object line: description Window > > update ports > > > > the object-group looks like this > > > > object-group service win-update-ports tcp > > description Window update ports > > port-object eq www > > port-object eq https > > > > > > I wrote a really terrible pix 7 config parser that works and does this > > stuff and wish I had found this module first. If I get a chance I'll > > see if I can make a patch for your module to make it work with names > > and descriptions, but I don't want to get in over my head.
> > > >
  Fri Nov 07 08:55:46 2008 The RT System itself - Status changed from 'rejected' to 'open'
  Wed Nov 12 12:52:53 2008 lifo101 [...] gmail.com - Correspondence added
PIX::Walker 1.10 was just uploaded to cpan. It'll take a bit for it to get indexed.
  Wed Nov 12 12:53:55 2008 lifo101 [...] gmail.com - Status changed from 'open' to 'resolved'