All of lore.kernel.org
 help / color / mirror / Atom feed
* Outstanding Coverity patches
@ 2013-12-17 18:38 Andrew Cooper
  2013-12-17 22:35 ` Matthew Daley
  2013-12-18 11:12 ` Ian Campbell
  0 siblings, 2 replies; 6+ messages in thread
From: Andrew Cooper @ 2013-12-17 18:38 UTC (permalink / raw)
  To: Xen-devel List; +Cc: Matthew Daley, Ian Jackson, Ian Campbell

Hello,

Following the reasonable complaint that "all of these Coverity fixes are
very similar and some are falling through the cracks", I decided that an
index email is in order, sumerising the outstanding patches and the
state I believe they are in.


* xl: Fix CHK_ERRNO()
    - 1386690317-11181-1-git-send-email-andrew.cooper3@citrix.com
    - Patch v3.  Acked but not committed

* tools/libxc: Fix error checking for xc_get_{cpu,node}map_size() callers
    - 1386776862-19921-1-git-send-email-andrew.cooper3@citrix.com
    - Quite a lot of arguing about the check being "<0" vs "<=0". 
Seeking maintainers executive decision.

* tools/libxl: Fix libxl__device_nic_from_xs_be()
    - 21140.47494.337790.858352@mariner.uk.xensource.com
    - Quite a bit of back and forth.  IanJ submitted a patch which I
have Reviewed-By.  Appears to have stagnated, despite several pings.


Matt: I cant recall if there are any of your fixes outstanding.  If
there are, can you list them please?

~Andrew

^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: Outstanding Coverity patches
  2013-12-17 18:38 Outstanding Coverity patches Andrew Cooper
@ 2013-12-17 22:35 ` Matthew Daley
  2013-12-18 10:57   ` Ian Campbell
  2013-12-18 11:12 ` Ian Campbell
  1 sibling, 1 reply; 6+ messages in thread
From: Matthew Daley @ 2013-12-17 22:35 UTC (permalink / raw)
  To: Andrew Cooper; +Cc: Ian Jackson, Ian Campbell, Xen-devel List

On Wed, Dec 18, 2013 at 7:38 AM, Andrew Cooper
<andrew.cooper3@citrix.com> wrote:
> Matt: I cant recall if there are any of your fixes outstanding.  If
> there are, can you list them please?

Nope, Ian J. recently committed them all. However, there's a query
that I'd still appreciate a reply to at
<CAD3Cane_2POAKWJ+5TrUYy6eEqqSOo+OLRO8qgbF5JsN+f3j-w@mail.gmail.com>.

- Matthew

^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: Outstanding Coverity patches
  2013-12-17 22:35 ` Matthew Daley
@ 2013-12-18 10:57   ` Ian Campbell
  2013-12-18 11:06     ` Matthew Daley
  0 siblings, 1 reply; 6+ messages in thread
From: Ian Campbell @ 2013-12-18 10:57 UTC (permalink / raw)
  To: Matthew Daley; +Cc: Andrew Cooper, Ian Jackson, Xen-devel List

On Wed, 2013-12-18 at 11:35 +1300, Matthew Daley wrote:
> On Wed, Dec 18, 2013 at 7:38 AM, Andrew Cooper
> <andrew.cooper3@citrix.com> wrote:
> > Matt: I cant recall if there are any of your fixes outstanding.  If
> > there are, can you list them please?
> 
> Nope, Ian J. recently committed them all. However, there's a query
> that I'd still appreciate a reply to at
> <CAD3Cane_2POAKWJ+5TrUYy6eEqqSOo+OLRO8qgbF5JsN+f3j-w@mail.gmail.com>.

gmane is having issues and neither marc.info nor bugs.xenproject.org
seem to know this message-id I'm afraid.

Ian.

^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: Outstanding Coverity patches
  2013-12-18 10:57   ` Ian Campbell
@ 2013-12-18 11:06     ` Matthew Daley
  2013-12-18 11:33       ` Ian Campbell
  0 siblings, 1 reply; 6+ messages in thread
From: Matthew Daley @ 2013-12-18 11:06 UTC (permalink / raw)
  To: Ian Campbell; +Cc: Andrew Cooper, Ian Jackson, Xen-devel List

On Wed, Dec 18, 2013 at 11:57 PM, Ian Campbell <Ian.Campbell@citrix.com> wrote:
> On Wed, 2013-12-18 at 11:35 +1300, Matthew Daley wrote:
>> On Wed, Dec 18, 2013 at 7:38 AM, Andrew Cooper
>> <andrew.cooper3@citrix.com> wrote:
>> > Matt: I cant recall if there are any of your fixes outstanding.  If
>> > there are, can you list them please?
>>
>> Nope, Ian J. recently committed them all. However, there's a query
>> that I'd still appreciate a reply to at
>> <CAD3Cane_2POAKWJ+5TrUYy6eEqqSOo+OLRO8qgbF5JsN+f3j-w@mail.gmail.com>.
>
> gmane is having issues and neither marc.info nor bugs.xenproject.org
> seem to know this message-id I'm afraid.
>
> Ian.
>

Bleh. The thread is "[PATCH 1/2] xenstore: sanity check incoming
message body lengths".

^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: Outstanding Coverity patches
  2013-12-17 18:38 Outstanding Coverity patches Andrew Cooper
  2013-12-17 22:35 ` Matthew Daley
@ 2013-12-18 11:12 ` Ian Campbell
  1 sibling, 0 replies; 6+ messages in thread
From: Ian Campbell @ 2013-12-18 11:12 UTC (permalink / raw)
  To: Andrew Cooper; +Cc: Matthew Daley, Ian Jackson, Xen-devel List

On Tue, 2013-12-17 at 18:38 +0000, Andrew Cooper wrote:
> * xl: Fix CHK_ERRNO()
> * tools/libxc: Fix error checking for xc_get_{cpu,node}map_size() callers
> * tools/libxl: Fix libxl__device_nic_from_xs_be()

All applied.

Ian

^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: Outstanding Coverity patches
  2013-12-18 11:06     ` Matthew Daley
@ 2013-12-18 11:33       ` Ian Campbell
  0 siblings, 0 replies; 6+ messages in thread
From: Ian Campbell @ 2013-12-18 11:33 UTC (permalink / raw)
  To: Matthew Daley; +Cc: Andrew Cooper, Ian Jackson, Xen-devel List

On Thu, 2013-12-19 at 00:06 +1300, Matthew Daley wrote:
> On Wed, Dec 18, 2013 at 11:57 PM, Ian Campbell <Ian.Campbell@citrix.com> wrote:
> > On Wed, 2013-12-18 at 11:35 +1300, Matthew Daley wrote:
> >> On Wed, Dec 18, 2013 at 7:38 AM, Andrew Cooper
> >> <andrew.cooper3@citrix.com> wrote:
> >> > Matt: I cant recall if there are any of your fixes outstanding.  If
> >> > there are, can you list them please?
> >>
> >> Nope, Ian J. recently committed them all. However, there's a query
> >> that I'd still appreciate a reply to at
> >> <CAD3Cane_2POAKWJ+5TrUYy6eEqqSOo+OLRO8qgbF5JsN+f3j-w@mail.gmail.com>.
> >
> > gmane is having issues and neither marc.info nor bugs.xenproject.org
> > seem to know this message-id I'm afraid.
> >
> > Ian.
> >
> 
> Bleh. The thread is "[PATCH 1/2] xenstore: sanity check incoming
> message body lengths".

Thanks, gmane is back and finds it:
http://article.gmane.org/gmane.comp.emulators.xen.devel/180941

Looks like one for Ian J to respond to.

Ian.

^ permalink raw reply	[flat|nested] 6+ messages in thread

end of thread, other threads:[~2013-12-18 11:33 UTC | newest]

Thread overview: 6+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2013-12-17 18:38 Outstanding Coverity patches Andrew Cooper
2013-12-17 22:35 ` Matthew Daley
2013-12-18 10:57   ` Ian Campbell
2013-12-18 11:06     ` Matthew Daley
2013-12-18 11:33       ` Ian Campbell
2013-12-18 11:12 ` Ian Campbell

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.