projects / qemu.git / commit
? search:
summary | shortlog | log | commit | commitdiff | tree
(parent: 40897c9) | patch
slirp: Clean up net_slirp_hostfwd_remove()'s use of get_str_sep()
authorMarkus Armbruster <armbru@redhat.com>
Wed, 16 Nov 2011 14:45:59 +0000 (15:45 +0100)
committerStefan Hajnoczi <stefanha@linux.vnet.ibm.com>
Tue, 22 Nov 2011 11:11:23 +0000 (11:11 +0000)
commite30e5eb6136aa3311dad4ca90f6546e05ae98f92
treeedf136192a0ba716ae2e4660f9d57c4f07be71b6tree | snapshot
parent40897c9c160393df922dfdb59cfa210048d3071dcommit | diff
slirp: Clean up net_slirp_hostfwd_remove()'s use of get_str_sep()

get_str_sep() can fail, but net_slirp_hostfwd_remove() doesn't check.
Works, because it initializes buf[] to "", which get_str_sep() doesn't
touch when it fails.  Coverity doesn't like it, and neither do I.

Change it to work exactly like slirp_hostfwd().

Acked-by: Jan Kiszka <jan.kiszka@siemens.com>
Signed-off-by: Markus Armbruster <armbru@redhat.com>
Signed-off-by: Stefan Hajnoczi <stefanha@linux.vnet.ibm.com>
net/slirp.c diff | blob | history
Unnamed repository; edit this file 'description' to name the repository.