Re: [krh/wayland/next 1/5] connection: Don't call memcpy with null pointer.

2012-10-16 Thread Kristian Høgsberg
On Fri, Oct 12, 2012 at 11:28:24AM +0200, John Kåre Alsaker wrote: Except where otherwise mentioned, all patches in the two patch series applied. Thanks! Kristian > --- > src/connection.c | 6 +++--- > 1 file changed, 3 insertions(+), 3 deletions(-) > > diff --git a/src/connection.c b/src/conn

Re: [krh/wayland/next 1/5] connection: Don't call memcpy with null pointer.

2012-10-16 Thread Kristian Høgsberg
On Fri, Oct 12, 2012 at 11:28:24AM +0200, John Kåre Alsaker wrote: > --- > src/connection.c | 6 +++--- > 1 file changed, 3 insertions(+), 3 deletions(-) > > diff --git a/src/connection.c b/src/connection.c > index 8497670..58b150b 100644 > --- a/src/connection.c > +++ b/src/connection.c > @@ -48

Re: [krh/wayland/next 1/5] connection: Don't call memcpy with null pointer.

2012-10-12 Thread John Kåre Alsaker
On Fri, Oct 12, 2012 at 3:05 PM, Rob Bradford wrote: > On 12 October 2012 10:28, John Kåre Alsaker > wrote: > >> - if (length > 0) >> + if (length > 0) { >> + memcpy(p, s, length); >> *sp =

Re: [krh/wayland/next 1/5] connection: Don't call memcpy with null pointer.

2012-10-12 Thread Rob Bradford
On 12 October 2012 10:28, John Kåre Alsaker wrote: > - if (length > 0) > + if (length > 0) { > + memcpy(p, s, length); > *sp = (const char *) p; > - else > +

[krh/wayland/next 1/5] connection: Don't call memcpy with null pointer.

2012-10-12 Thread John Kåre Alsaker
--- src/connection.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/src/connection.c b/src/connection.c index 8497670..58b150b 100644 --- a/src/connection.c +++ b/src/connection.c @@ -486,12 +486,12 @@ wl_closure_vmarshal(struct wl_object *sender,