From mboxrd@z Thu Jan 1 00:00:00 1970 From: Ian Campbell Subject: Re: [PATCH for-4.5 0/3] Coverity fixes for python lowlevel libraries Date: Fri, 28 Nov 2014 12:07:12 +0000 Message-ID: <1417176432.23604.23.camel@citrix.com> References: <1417091674-8163-1-git-send-email-andrew.cooper3@citrix.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <1417091674-8163-1-git-send-email-andrew.cooper3@citrix.com> List-Unsubscribe: , List-Post: List-Help: List-Subscribe: , Sender: xen-devel-bounces@lists.xen.org Errors-To: xen-devel-bounces@lists.xen.org To: Andrew Cooper Cc: Wei Liu , Ian Jackson , Xen Coverity Team , Xen-devel List-Id: xen-devel@lists.xenproject.org On Thu, 2014-11-27 at 12:34 +0000, Andrew Cooper wrote: > While Xend is certainly dead and gone, XenServer at the very least still has > consumers of these python libraries. AIUI XS mainly uses the xs.c stuff, with the xc.c stuff being mainly debug utilities. This is important because while the xs.c is simple and obviously correct the xc.c patches are a little more involved. > > Konrad: I am requesting a release ack for this. All 5 issues are bugs with > the handling of error cases, rather than with the basic functionality > provided. With these changes, Coverity is of the opinion that the python > libraries are perfect (0 issues), and I feel this is a worthy position to be > in for 4.5 > > Andrew Cooper (3): > python/xc: Fix multiple issues in pyflask_context_to_sid() > python/xc: Fix multiple issues in pyxc_readconsolering() > python/xs: Correct the indirection of the NULL xshandle() check > > tools/python/xen/lowlevel/xc/xc.c | 34 +++++++++------------------------- > tools/python/xen/lowlevel/xs/xs.c | 2 +- > 2 files changed, 10 insertions(+), 26 deletions(-) >