Bugs item #1991343, was opened at 2008-06-12 00:12
Message generated for change (Comment added) made by shammash
You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=611248&aid=1991343&group_id=95403

Please note that this message will contain a full copy of the comment thread,
including the initial issue submission, for this request,
not just the latest update.
Category: None
Group: None
>Status: Closed
>Resolution: Fixed
Priority: 5
Private: No
Submitted By: Luca Bigliardi (shammash)
>Assigned to: renzo davoli (rd235)
Summary: slirpvde: poor error handling

Initial Comment:
Forwarding from debian BTS #460734.

As slirpvde will be rewritten with lwip6 this
should disappear.

Hello again,

I have more information for this 'bug'. Well, it's 
actually not the bug I thought, since the error is due 
to poor error handling. In my experimentations with the 
package, the command line I used to use was wrong, hence 
the Bus error. Looking at the source code it looks like 
he main problems are because a lot of error checking 
(even as innocent as File not found) is not done, hence  
carrying around NULL pointers. In particular the problem 
was after vde_open in slirpvde.c (around line 429) which 
doesn't check if the open was succesfull, and after 
vde_datafd (around line 443) where the return value is 
not check (in this case the previous open failed).



thanks
graziano



----------------------------------------------------------------------

>Comment By: Luca Bigliardi (shammash)
Date: 2010-07-03 16:47

Message:
newer slirp improves error handling

----------------------------------------------------------------------

You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=611248&aid=1991343&group_id=95403

------------------------------------------------------------------------------
This SF.net email is sponsored by Sprint
What will you do first with EVO, the first 4G phone?
Visit sprint.com/first -- http://p.sf.net/sfu/sprint-com-first
_______________________________________________
vde-users mailing list
vde-users@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/vde-users

Reply via email to