On 11/17/2017 04:07 AM, Stefan Weil wrote:
Am 16.11.2017 um 03:28 schrieb Mao Zhongyi:
Cc: Peter Maydell <peter.mayd...@linaro.org>
Cc: Jason Wang <jasow...@redhat.com>
Cc: Zhang Chen <zhangc...@gmail.com>
Cc: Li Zhijian <lizhij...@cn.fujitsu.com>
Cc: Paolo Bonzini <pbonz...@redhat.com>
Fixes: 8ec14402029d783720f4312ed8a925548e1dad61
Reported-by: Peter Maydell <peter.mayd...@linaro.org>
Reported-by: Paolo Bonzini <pbonz...@redhat.com>
Signed-off-by: Mao Zhongyi <maozy.f...@cn.fujitsu.com>
---
net/colo-compare.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/net/colo-compare.c b/net/colo-compare.c
index ccdcba2..1ce195f 100644
--- a/net/colo-compare.c
+++ b/net/colo-compare.c
@@ -179,7 +179,7 @@ static int packet_enqueue(CompareState *s, int mode,
Connection **con)
"drop packet");
}
}
- con = &conn;
+ *con = conn;
return 0;
}
Hi, Stefan
The patch is definitely needed if the new function parameter con
should work.
It also fixes a gcc compiler warning:
net/colo-compare.c:139:67: warning: parameter ‘con’ set but not used
[-Wunused-but-set-parameter]
I think using -Wextra would be really good to catch such bugs earlier.
Thanks a bunch. I got it. :)
Mao