Debian Bug report logs - #484295
vde2: vde_switch segfaults on vlan related CLI commands on AMD64

Toggle useless messages

Message #5 received at submit@bugs.debian.org (full text, mbox, reply):

From: Bjørn Mork <bjorn@mork.no>

To: Debian Bug Tracking System <submit@bugs.debian.org>

Subject: vde2: vde_switch segfaults on vlan related CLI commands on AMD64

Date: Tue, 03 Jun 2008 16:01:20 +0200

Package: vde2
Version: 2.2.0-pre2-1
Severity: important

-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1

I can consistently make vde_switch segfault by using a CLI sequence like this:

 bjorn@canardo:~$ vde_switch 
 
 vde$ vlan/create 42
 1000 Success
 
 vde$ vlan/allprint
 Segmentation fault


There are many other combinations which will achieve the same result.  The effect
is that it is impossible to configure the virtual switches with any non-default
vlans.  Using a configuration file at switch startup has the same crashing effect.

The switch does still function as expected for a non-managed switch, so this bug
does not render it completely useless.  But the usability is severely limited.

I've tested on an i386 system, and cannot reproduce the bug there.  Given the 
implemantions dependency on the sizeof(long), I assume this makes it a 64bit
bug.  Will attempt to fix it and provide a patch if I can.  Just wanted to register
this bug first in case someone is working on it, or someone can point me to
some configuration error on my side.



Bjorn



- -- System Information:
Debian Release: lenny/sid
  APT prefers stable
  APT policy: (700, 'stable'), (650, 'testing')
Architecture: amd64 (x86_64)

Kernel: Linux 2.6.24-1-amd64 (SMP w/4 CPU cores)
Locale: LANG=en_US, LC_CTYPE=en_US (charmap=ISO-8859-1)
Shell: /bin/sh linked to /bin/bash

Versions of packages vde2 depends on:
ii  adduser                     3.107        add and remove users and groups
ii  libc6                       2.7-10       GNU C Library: Shared libraries
ii  libpcap0.8                  0.9.5-1      System interface for user-level pa
ii  libvdemgmt0                 2.2.0-pre2-1 Virtual Distributed Ethernet - Man
ii  libvdeplug2                 2.2.0-pre2-1 Virtual Distributed Ethernet - Plu

Versions of packages vde2 recommends:
ii  daemon                        0.6.3-1    turns other processes into daemons

- -- no debconf information

-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.6 (GNU/Linux)

iD8DBQFIRU6w10rqkowbIskRAjjtAJ0RLsspureF2/GE66BzpHPwgHN4HgCfYm2A
ytL92nzUHAlIT5k/4vqkrrA=
=lbnd
-----END PGP SIGNATURE-----

Message #10 received at 484295@bugs.debian.org (full text, mbox, reply):

From: Bjørn Mork <bjorn@mork.no>

To: 484295@bugs.debian.org

Subject: [patch] vde2: vde_switch segfaults on vlan related CLI commands on AMD64

Date: Wed, 04 Jun 2008 14:00:34 +0200

[Message part 1 (text/plain, inline)]

tags 484295 patch
thanks

Yuckety.  There wasn't much of any NULL pointer checking or anything in
this code.  The segfaults were caused by vlanprintelem() trying to dump
the contents of vlant[vlan].table for non-existing VLANs.  This should
of course never happen, but when it did a simple test for
vlant[vlan].table == NULL could have easily made it into an error
message instead of a segfault.

Well, I'm not going to fix all that now, given that there must be
hundreds of places to add such tests all over the place.

The main reason why everything went wrong on 64bit systems were lots of
type mismatches in bitarray.h.  Among the more serious was BAC_SET using
a signed int as temporary storage for bitarray elemens, causing
interesting effects if you happened to set bit 31, e.g. by creating VLAN
31...

There were also a number of attempts to shift 1 more than 31 bits to the
left, which doesn't really work well.

The attached patch fixes these important parts and stabilizes vde_switch
on my system.  Please consider applying it before lenny is released.

Thanks,
Bjørn

[bitarray.diff (text/x-diff, attachment)]

Message #17 received at 484295@bugs.debian.org (full text, mbox, reply):

From: Bjørn Mork <bjorn@mork.no>

To: 484295@bugs.debian.org

Subject: Bug#484295: fixed upstream (vde2: vde_switch segfaults on vlan related CLI commands on AMD64)

Date: Thu, 12 Jun 2008 08:59:24 +0200

tags 484295 + fixed-upstream
thanks

This bug was fixed by upstream in svn r268.  Please consider either
applying the patch or merging svn r268 or newer before the release of
lenny, as this bug really degrades the usability of the package on 64bit
systems.


Thanks,
Bjorn

Message #24 received at 484295@bugs.debian.org (full text, mbox, reply):

From: Filippo Giunchedi <filippo@debian.org>

To: control@bugs.debian.org

Cc: 469098@bugs.debian.org, 484295@bugs.debian.org

Subject: setting package to libvdemgmt0 libvdeplug2 vde2-cryptcab vde libvdeplug2-dev vde2 libvdemgmt0-dev ...

Date: Sun, 15 Jun 2008 18:45:23 +0200

# Automatically generated email from bts, devscripts version 2.10.29
#
# vde2 (2.2.1-1) UNRELEASED; urgency=low
#
#  * New upstream release
#    - fix vlan commands on amd64 (Closes: #484295)
#    - fix mac addresses switch between ports (Closes: #469098)
#

package libvdemgmt0 libvdeplug2 vde2-cryptcab vde libvdeplug2-dev vde2 libvdemgmt0-dev
tags 484295 + pending
tags 469098 + pending

Message #31 received at 484295-close@bugs.debian.org (full text, mbox, reply):

From: Filippo Giunchedi <filippo@debian.org>

To: 484295-close@bugs.debian.org

Subject: Bug#484295: fixed in vde2 2.2.1-1

Date: Tue, 17 Jun 2008 17:47:04 +0000

Source: vde2
Source-Version: 2.2.1-1

We believe that the bug you reported is fixed in the latest version of
vde2, which is due to be installed in the Debian FTP archive:

libvdemgmt0-dev_2.2.1-1_amd64.deb
  to pool/main/v/vde2/libvdemgmt0-dev_2.2.1-1_amd64.deb
libvdemgmt0_2.2.1-1_amd64.deb
  to pool/main/v/vde2/libvdemgmt0_2.2.1-1_amd64.deb
libvdeplug2-dev_2.2.1-1_amd64.deb
  to pool/main/v/vde2/libvdeplug2-dev_2.2.1-1_amd64.deb
libvdeplug2_2.2.1-1_amd64.deb
  to pool/main/v/vde2/libvdeplug2_2.2.1-1_amd64.deb
vde2-cryptcab_2.2.1-1_amd64.deb
  to pool/main/v/vde2/vde2-cryptcab_2.2.1-1_amd64.deb
vde2_2.2.1-1.diff.gz
  to pool/main/v/vde2/vde2_2.2.1-1.diff.gz
vde2_2.2.1-1.dsc
  to pool/main/v/vde2/vde2_2.2.1-1.dsc
vde2_2.2.1-1_amd64.deb
  to pool/main/v/vde2/vde2_2.2.1-1_amd64.deb
vde2_2.2.1.orig.tar.gz
  to pool/main/v/vde2/vde2_2.2.1.orig.tar.gz
vde_2.2.1-1_all.deb
  to pool/main/v/vde2/vde_2.2.1-1_all.deb



A summary of the changes between this version and the previous one is
attached.

Thank you for reporting the bug, which will now be closed.  If you
have further comments please address them to 484295@bugs.debian.org,
and the maintainer will reopen the bug report if appropriate.

Debian distribution maintenance software
pp.
Filippo Giunchedi <filippo@debian.org> (supplier of updated vde2 package)

(This message was generated automatically at their request; if you
believe that there is a problem with it please contact the archive
administrators by mailing ftpmaster@debian.org)


-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1

Format: 1.8
Date: Tue, 17 Jun 2008 15:36:32 +0200
Source: vde2
Binary: vde2 vde vde2-cryptcab libvdeplug2-dev libvdeplug2 libvdemgmt0-dev libvdemgmt0
Architecture: source amd64 all
Version: 2.2.1-1
Distribution: unstable
Urgency: low
Maintainer: Debian VSquare Team <pkg-vsquare-devel@lists.alioth.debian.org>
Changed-By: Filippo Giunchedi <filippo@debian.org>
Description: 
 libvdemgmt0 - Virtual Distributed Ethernet - Management console library
 libvdemgmt0-dev - Virtual Distributed Ethernet - Mgmt console development files
 libvdeplug2 - Virtual Distributed Ethernet - Plug library
 libvdeplug2-dev - Virtual Distributed Ethernet - Plug development files
 vde        - transitional dummy package which can be safely removed
 vde2       - Virtual Distributed Ethernet
 vde2-cryptcab - Virtual Distributed Ethernet - CryptCab
Closes: 466363 469098 484295
Changes: 
 vde2 (2.2.1-1) unstable; urgency=low
 .
   * New upstream release
     - fix vlan commands on amd64 (Closes: #484295)
     - fix mac addresses switch between ports (Closes: #469098)
   * Suggest: qemu and kvm as requested in #461514
   * Expand and spell-check README.Debian, add manual method example
     (Closes: #466363)
   * Do not assume MAKEDEV presence in postinst
   * Remove /usr/bin/daemon usage from ifupdown scripts (and Recommends)
   * Add manpage for vde_tunctl
   * Upgrade to S-V 3.8.0 (add Homepage field)
Checksums-Sha1: 
 4f16a0602a5fccb780bd8d565c91bcfa55a59add 1377 vde2_2.2.1-1.dsc
 86aba07f3a9dd5c0906433d80ae6acf3755d3e61 658805 vde2_2.2.1.orig.tar.gz
 6f4239e67235746abfc94a57868bcfd79d4ebf2e 11910 vde2_2.2.1-1.diff.gz
 bc3c8d096558d76640eccb8a81c55d1a28842d63 198746 vde2_2.2.1-1_amd64.deb
 01067d0bbe256ebfa79d63bbb886424010d7ef94 4524 vde_2.2.1-1_all.deb
 371c355087337f047bd9745a722e03f72db4d2a7 16290 vde2-cryptcab_2.2.1-1_amd64.deb
 09d00c47fbeade6135280885dd84fd0abf8194ca 12366 libvdeplug2-dev_2.2.1-1_amd64.deb
 8becd5eefa8ce4196dddce2c506ec341aad4349c 11352 libvdeplug2_2.2.1-1_amd64.deb
 5ad3f86cd3754a984e1ccb31baecc393ada2ec29 13568 libvdemgmt0-dev_2.2.1-1_amd64.deb
 49690760f5f006d3b1dbff91b07a548333a69e36 12466 libvdemgmt0_2.2.1-1_amd64.deb
Checksums-Sha256: 
 5ea7d654d343b39732f7b1710c9eb4d04112850e796f6592c2a460ca88dde872 1377 vde2_2.2.1-1.dsc
 8908ad37457fd0c49524844ac1a713410daae55ac8aa286a6d59df0428fc83bb 658805 vde2_2.2.1.orig.tar.gz
 8970f6e79aaaed9b8a7131c50da2e11923a9ee99162de05277698a3dc2244945 11910 vde2_2.2.1-1.diff.gz
 cac52ed791f34222d0b4d2157da9beb79884855cd476115751f1078d8bfdbe9c 198746 vde2_2.2.1-1_amd64.deb
 7196834d512e7f88ab35b8706f8185ac6877f3bca814b26674f8640f96cc073f 4524 vde_2.2.1-1_all.deb
 bd982c84608d49b395eca308ae35bc60dd1dca4352ffe1b5d8cb9fd876687250 16290 vde2-cryptcab_2.2.1-1_amd64.deb
 379f9c75b7a68619938253a2e4747bc359e75bfdd2ac48a00e3deb7a28a25a58 12366 libvdeplug2-dev_2.2.1-1_amd64.deb
 95737f58867e825a2c1187ce3a7fec08cc4d52e9a03bd22e30ae4c0e71072f4e 11352 libvdeplug2_2.2.1-1_amd64.deb
 aceebb742dc72cda7d31d850a7016e127838ec99eb2471fad31052c97040c226 13568 libvdemgmt0-dev_2.2.1-1_amd64.deb
 a1d6f184d465a492954131cec31f6145b2817eba49849ea3ce216d0eb901788d 12466 libvdemgmt0_2.2.1-1_amd64.deb
Files: 
 daabaa9178c55a2bd888c89766cf38d4 1377 net optional vde2_2.2.1-1.dsc
 682489504dbbf4acbf332de7b739078f 658805 net optional vde2_2.2.1.orig.tar.gz
 96d8fa983d4732f0aaace2e4ab5d01f0 11910 net optional vde2_2.2.1-1.diff.gz
 74f80001a55d9415fceba82e6a4ecac2 198746 net optional vde2_2.2.1-1_amd64.deb
 fef7925a1a16f42ed8d006eae44328a8 4524 net optional vde_2.2.1-1_all.deb
 2690e7eb19c3f3f7af440d9eb9a7d391 16290 net optional vde2-cryptcab_2.2.1-1_amd64.deb
 259891517af3db2d44ba8a0611aa5788 12366 libdevel optional libvdeplug2-dev_2.2.1-1_amd64.deb
 674c41c3fe5d00da0f3862287917e04d 11352 libs optional libvdeplug2_2.2.1-1_amd64.deb
 6519abc7b7d51b6a07f77b107f96bc1e 13568 libdevel optional libvdemgmt0-dev_2.2.1-1_amd64.deb
 21a7329fdef3cc200a917e3f008c6af7 12466 libs optional libvdemgmt0_2.2.1-1_amd64.deb

-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.9 (GNU/Linux)

iEYEARECAAYFAkhX9YcACgkQABzeamt51AGzhwCeP2KQt+r+sivLA2MeLxaTl2M5
KpwAn0QPBg21uoMhA9T4CzMzv/COi80i
=GsL0
-----END PGP SIGNATURE-----

Send a report that this bug log contains spam.