The code now now matches its comment, and will actually catch the case of a bad xs handle.
Signed-off-by: Andrew Cooper <andrew.coop...@citrix.com> Coverity-ID: 1055948 CC: Ian Campbell <ian.campb...@citrix.com> CC: Ian Jackson <ian.jack...@eu.citrix.com> CC: Wei Liu <wei.l...@citrix.com> CC: Xen Coverity Team <cover...@xen.org> --- tools/python/xen/lowlevel/xs/xs.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/tools/python/xen/lowlevel/xs/xs.c b/tools/python/xen/lowlevel/xs/xs.c index 84e1711..ec364bb 100644 --- a/tools/python/xen/lowlevel/xs/xs.c +++ b/tools/python/xen/lowlevel/xs/xs.c @@ -816,7 +816,7 @@ static int parse_transaction_path(XsHandle *self, PyObject *args, *xh = xshandle(self); - if (!xh) + if (!*xh) return 0; if (!PyArg_ParseTuple(args, "ss", &thstr, path)) -- 1.7.10.4 _______________________________________________ Xen-devel mailing list Xen-devel@lists.xen.org http://lists.xen.org/xen-devel