[Qemu-devel] [PATCH] slirp: Fix typo in net_slirp_hostfwd_remove

[Qemu-devel] [PATCH] slirp: Fix typo in net_slirp_hostfwd_remove

[Geoffrey Thomas]

Report an error when err is nonzero, not when it is zero.

Signed-off-by: Geoffrey Thomas <geofft@ldpreload.com>
---
  net/slirp.c |    2 +-
  1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/net/slirp.c b/net/slirp.c
index 6646ecb..18e07ba 100644
--- a/net/slirp.c
+++ b/net/slirp.c
@@ -351,7 +351,7 @@ void net_slirp_hostfwd_remove(Monitor *mon, const QDict *qdict)
                                 host_addr, host_port);

      monitor_printf(mon, "host forwarding rule for %s %s\n", src_str,
-                   err ? "removed" : "not found");
+                   err ? "not found" : "removed");
      return;

   fail_syntax:
-- 
1.7.7.3

Re: [Qemu-devel] [PATCH] slirp: Fix typo in net_slirp_hostfwd_remove

[Jan Kiszka]

[-- Attachment #1: Type: text/plain, Size: 832 bytes --]

On 2011-12-17 10:23, Geoffrey Thomas wrote:
> Report an error when err is nonzero, not when it is zero.
> 
> Signed-off-by: Geoffrey Thomas <geofft@ldpreload.com>
> ---
>  net/slirp.c |    2 +-
>  1 files changed, 1 insertions(+), 1 deletions(-)
> 
> diff --git a/net/slirp.c b/net/slirp.c
> index 6646ecb..18e07ba 100644
> --- a/net/slirp.c
> +++ b/net/slirp.c
> @@ -351,7 +351,7 @@ void net_slirp_hostfwd_remove(Monitor *mon, const
> QDict *qdict)
>                                 host_addr, host_port);
> 
>      monitor_printf(mon, "host forwarding rule for %s %s\n", src_str,
> -                   err ? "removed" : "not found");
> +                   err ? "not found" : "removed");
>      return;
> 
>   fail_syntax:

Oops, obviously. Stefan, should be another trivial one for you.

Thanks,
Jan


[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 262 bytes --]

Re: [Qemu-devel] [PATCH] slirp: Fix typo in net_slirp_hostfwd_remove

[Stefan Hajnoczi]

On Sat, Dec 17, 2011 at 04:23:59AM -0500, Geoffrey Thomas wrote:
> Report an error when err is nonzero, not when it is zero.
> 
> Signed-off-by: Geoffrey Thomas <geofft@ldpreload.com>
> ---
>  net/slirp.c |    2 +-
>  1 files changed, 1 insertions(+), 1 deletions(-)

Thanks, applied to the trivial patches tree:
http://repo.or.cz/w/qemu/stefanha.git/shortlog/refs/heads/trivial-patches

Stefan