linux emulation COMPAT_43 removal part 1
David P. Reese, Jr.
daver at gomerbud.com
Sat Sep 6 15:37:41 PDT 2003
On Sat, Sep 06, 2003 at 01:38:12PM -0700, Matthew Dillon wrote:
> David, I reviewed your patch and I thought I found a bug, but it turns
> out the bug is in the original API that you duplicated exactly!
> getsockaddr() (and consequentially linux_getsockaddr()) was not checking
> for a passed length parameter that was too small. This can result in
> a 0-length malloc and then an attempt to assign sa->sa_len.
Good catch. I was hoping that I would be able to avoid introducing bugs
by duplicating the behavior of the old code. Guess I was wrong.
> Also, I looked at the FreeBSD-current code and they appear to be using
> kern_*(), so if we adopt kern_*() we will be using a comparable API.
This naming convention does look prettier anyway. I've got a another patch
for uipc_syscalls.c and linux_socket.c on the way. I'll make sure to
change the split syscall names before I submit.
> In anycase, don't worry about the malloc problem, I will commit a check
> along with your patch set.
>
> -Matt
Thanks a bunch.
--
David P. Reese, Jr. daver at xxxxxxxxxxxx
http://www.gomerbud.com/daver/
More information about the Submit
mailing list