linux emulation COMPAT_43 removal part 1
    Matthew Dillon 
    dillon at apollo.backplane.com
       
    Sat Sep  6 13:38:40 PDT 2003
    
    
  
    oops, resending. I manged David's email address.
:>     base, but I could go either way.  e.g. kern_bind() instead of bind1().
:>     Definite not _syscall_bind() (ick).  But before we make that sort of
:>     change I will commit the linux patch using the current naming 
:>     convention.
:
:	Deal!  If I get some time in the next two weeks, I will convert
:	as many syscalls as I can into the split system, and then
:	collectively change their name from blah1() to kern_blah().
:
:	Regards,
:
:		- Hiten (hmp at xxxxxxxxxxxxx)
    Woa, timeout!  Coordinate with David Reese, he's been working on
    splitting the system calls as well.  There are a lot of syscalls so
    the help is appreciated, I suggest that you guys divvy up the files
    as you go.
    -
    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.
    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.
    In anycase, don't worry about the malloc problem, I will commit a check
    along with your patch set.
					-Matt
					Matthew Dillon 
					<dillon at xxxxxxxxxxxxx>
    
    
More information about the Submit
mailing list