Silently fail with wrong vde socket dir

Bug #676029 reported by Étienne BERSAC
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
QEMU
Fix Released
Undecided
Unassigned

Bug Description

Hi,

Using qemu 0.12.5, kvm silently fail with exit code 1 when using -net vde and a wrong path for sock. Actually, the sock option is mean to be the socket dir of the vde_switch, not the socket itself.

With -net vde,sock=/var/run/vde/vde0/ctl , strace ends with the following messages :

connect(7, {sa_family=AF_FILE, path="/var/run/vde/vde0/ctl/ctl"}, 110) = -1 ENOTDIR (Not a directory)
close(7) = 0
close(8) = 0
exit_group(1) = ?
root ~#

Please add a meaningful message.

Regards,
Étienne

Revision history for this message
Jes Sorensen (jes-sorensen) wrote :

Hello,

Could you please provide more data, what kinda of system and version are you running on?

Jes

Revision history for this message
Michael Tokarev (mjt+launchpad-tls) wrote :

There's no need to add any more specific information. The bug's in the code in qemu.

net/vde.c:

static int net_vde_init(VLANState *vlan, const char *model,
                        const char *name, const char *sock,
                        int port, const char *group, int mode)
{
...
    vde = vde_open(init_sock, (char *)"QEMU", &args);
    if (!vde){
        return -1;
    }
...
}

There's no message generated there. Callers merely pass the failure up the road, where it's finally handled as exit(1). If _anything_ is wrong in vde_open() (which can fail due to variety of reasons, including wrong path to the listening socket and what not), nothing will indicate that, just a trivial, silent exit.

Changed in qemu:
status: New → Confirmed
Revision history for this message
Jes Sorensen (jes-sorensen) wrote :

Given that you know what the problem is, it would probably have been faster to post a patch than just updating the bug and marking it confirmed....

Revision history for this message
Thomas Huth (th-huth) wrote :

A fix for this problem has finally been contributed here:
https://git.qemu.org/?p=qemu.git;a=commitdiff;h=7587855cd23755a7a6bd

Changed in qemu:
status: Confirmed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.