On Thu, Nov 16, 2017 at 10:28:32AM +0800, Mao Zhongyi wrote:
Cc: Peter Maydell <[email protected]>
Cc: Jason Wang <[email protected]>
Cc: Zhang Chen <[email protected]>
Cc: Li Zhijian <[email protected]>
Cc: Paolo Bonzini <[email protected]>
Fixes: 8ec14402029d783720f4312ed8a925548e1dad61
Reported-by: Peter Maydell <[email protected]>
Reported-by: Paolo Bonzini <[email protected]>
Signed-off-by: Mao Zhongyi <[email protected]>
Code-wise, this looks like a valid fix to the existing code.
Reviewed-by: Darren Kenny <[email protected]>
But testing wise, have you confirmed that things are behaving as you
expected with the previous patch, since previously when calling
colo_compare_connection(), the value of conn would have always been
its initialized value of NULL.
Just want to be sure that fixing this doesn't end up breaking your
expected behaviour given that all your testing before would have had
a NULL value in conn.
Thanks,
Darren.
---
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;
}
--
2.9.4
