Re: BUG #4433: entries like "host all all 10.0.50.31/0 ..." should not be allowed or trigger a warning

From: Craig Ringer <craig(at)postnewspapers(dot)com(dot)au>
To: "security improvement proposal: pg_hba(dot)conf and CIDR mask" <marc(at)intershop(dot)de>
Cc: pgsql-bugs(at)postgresql(dot)org
Subject: Re: BUG #4433: entries like "host all all 10.0.50.31/0 ..." should not be allowed or trigger a warning
Date: 2008-09-23 10:50:49
Message-ID: 48D8CA09.3080609@postnewspapers.com.au
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-bugs

security improvement proposal: pg_hba.conf and CIDR mask wrote:
> The following bug has been logged online:
>
> Bug reference: 4433
> Logged by: security improvement proposal: pg_hba.conf and CIDR
> mask
> Email address: marc(at)intershop(dot)de
> PostgreSQL version: 8.2.4
> Operating system: Linux
> Description: entries like "host all all 10.0.50.31/0 ..."
> should not be allowed or trigger a warning
> Details:
>
> Hello,
>
> not really a bug, but a possible security issue for wrongly configured
> installations.
>
> A CIDR mask length of 0 will allow to connect from any location. I did this
> mistake as I didn't read the documentation carefully enough.
>
> Checking the mask against the IP address would prevent such errors:
>
> /0 : disallow ?

0.0.0.0/0 should continue to be accepted.

> /24 : IP must ends with .0
> /16 : IP must ends with .0.0

If you're going to do that, you might as well enforce it for any CIDR
subnet and say that the address given must be the network address, not a
host address within the network. That way it works for non-multiple-of-8
CIDR subnets too.

--
Craig Ringer

In response to

Browse pgsql-bugs by date

  From Date Subject
Next Message Dean Rasheed 2008-09-23 14:31:14 BUG #4434: Error inserting into view - unrecognized node type: 313
Previous Message security improvement proposal: pg_hba.conf and CIDR mask 2008-09-23 09:44:24 BUG #4433: entries like "host all all 10.0.50.31/0 ..." should not be allowed or trigger a warning