[prev in list] [next in list] [prev in thread] [next in thread]
List: openssh-bugs
Subject: [Bug 1665] New: prefix_pton is inefficient
From: bugzilla-daemon () bugzilla ! mindrot ! org
Date: 2009-10-26 11:07:30
Message-ID: bug-1665-705 () http ! bugzilla ! mindrot ! org/
[Download RAW message or body]
https://bugzilla.mindrot.org/show_bug.cgi?id=1665
Summary: prefix_pton is inefficient
Product: py-radix
Version: -current
Platform: ix86
OS/Version: Linux
Status: NEW
Severity: enhancement
Priority: P2
Component: Default
AssignedTo: unassigned-bugs@mindrot.org
ReportedBy: weinholt@csbnet.se
Created an attachment (id=1710)
--> (https://bugzilla.mindrot.org/attachment.cgi?id=1710)
change prefix_pton to use inet_pton()
py-radix 0.5 uses getaddrinfo() to parse IPv4/IPv6 addresses. On
GNU/Linux this procedure creates a socket and performs a few syscalls
on it. It would be better to parse addresses without performing
syscalls.
I noticed this when writing a program that generates DHCP configuration
for ~15k hosts. When running the program on the server running the DHCP
daemon it ran out of some sort of kernel memory (NFS stopped working,
neighbor table filled, new TCP connections failed). For some reason the
development machine didn't have the same problem. I traced it down to
the getaddrinfo() call.
To solve this problem I patched py-radix to use inet_pton instead.
Please review the attached patch. Also, the testbed runs in 3.1 seconds
when it previously took 4.3s.
The patch also fixes the error messages. Previously invalid prefix
lengths would result in a garbage error message, and a failed call to
getaddrinfo() would return a pointer to inside the function's stack.
--
Configure bugmail: https://bugzilla.mindrot.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are watching the assignee of the bug.
_______________________________________________
openssh-bugs mailing list
openssh-bugs@mindrot.org
https://lists.mindrot.org/mailman/listinfo/openssh-bugs
[prev in list] [next in list] [prev in thread] [next in thread]
Configure |
About |
News |
Add a list |
Sponsored by KoreLogic