Project

General

Profile

Actions

Bug #803

closed

Fix a bug in route(8)

Added by nant over 16 years ago. Updated over 16 years ago.

Status:
Closed
Priority:
Normal
Assignee:
-
Category:
-
Target version:
-
Start date:
Due date:
% Done:

0%

Estimated time:

Description

Hi,

The attached patch allows "route add 128/8 <gateway>" to work
correctly. While here, rectify the way netmasks are deduced from the
address class.

Please verify.

Noticed-by: Tom Judge <>
Reported-on: freebsd-net mailing list

Thanks,
Nuno


Files

sbin.route.patch (1.38 KB) sbin.route.patch nant, 09/09/2007 03:27 PM
netstat-cidr.patch (1.14 KB) netstat-cidr.patch nant, 09/10/2007 05:43 PM
Actions #1

Updated by dillon over 16 years ago

:Hi,
:
:The attached patch allows "route add 128/8 <gateway>" to work
:correctly. While here, rectify the way netmasks are deduced from the
:address class.
:
:Please verify.
:
:Noticed-by: Tom Judge <>
:Reported-on: freebsd-net mailing list
:
:Thanks,
:Nuno

Works here.  I noticed something while testing it... netstat -rn
(display routes) is not displaying CIDR blocks properly. Look:

Internet:
Destination Gateway Flags Refs Use Netif Expire
127.0.0.1 127.0.0.1 UH 0 183 lo0
128.0/8 216.240.41.5 UGSc 0 0 nfe0

That should read 128/8, not 128.0/8.  It's mixing the core classes
with the CIDR specification.
Would you like to have a go at fixing that, too?
If nobody reports any bugs I'll commit your route patch on Monday.
-Matt
Actions #2

Updated by nant over 16 years ago

Yeah, I've seen that too. I think that netstat should be teached
better regarding CIDR. I'll have a look at it during this week.

Thanks,
Nuno

Actions #3

Updated by nant over 16 years ago

Hi,

I think this fixes netstat.

Thanks,
Nuno

Actions #4

Updated by sepherosa over 16 years ago

Committed. Thanks!

Best Regards,
sephe

Actions

Also available in: Atom PDF