[PATCH wayland 3/6] server: Split out code to initialize the socket address for a display name

[Pekka Paalanen]

On Thu, 7 Aug 2014 15:41:10 +0300
Pekka Paalanen <ppaalanen at gmail.com> wrote:

> On Thu, 17 Jul 2014 13:54:21 -0400
> "Jasper St. Pierre" <jstpierre at mecheye.net> wrote:
> 
> > We'll use this to autodetect a good socket to open on.
> > ---
> >  src/wayland-server.c | 41 ++++++++++++++++++++++++++---------------
> >  1 file changed, 26 insertions(+), 15 deletions(-)
> 
> 79b1d2039aeb77b712cf4e1bb4049ebf9c453b59 is the first bad commit
> commit 79b1d2039aeb77b712cf4e1bb4049ebf9c453b59
> Author: Jasper St. Pierre <jstpierre at mecheye.net>
> Date:   Thu May 8 10:25:13 2014 -0400
> 
>     server: Split out code to initialize the socket address for a display name
>     
>     We'll use this to autodetect a good socket to open on.
> 
> 
> This patch breaks 'make check' on socket-test.
> 
> error: socket path "/home/pq/local/tmp/aaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaa" plus null terminator exceeds 108 bytes
> socket-test: tests/socket-test.c:79: socket_path_overflow_server_create: Assertion `(*__errno_location ()) == 36' failed.
> test "socket_path_overflow_server_create":      signal 6, fail.
> 

And this is now fixed with Marek Chalupa's patch that was already on
the list. Thanks for pointing that out, Jasper.
- pq