[systemd-devel] [PATCH] core: do not make sockets dependent on lo

Lennart Poettering lennart at poettering.net
Tue Jan 15 12:01:02 PST 2013


On Sat, 12.01.13 23:05, Michał Bartoszkiewicz (mbartoszkiewicz at gmail.com) wrote:

> /sys/subsystem/net/devices/lo is never considered active, so sockets
> with BindToDevice=lo would never be activated.

Makes sense. Applied.

> ---
>  src/core/socket.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/src/core/socket.c b/src/core/socket.c
> index fcbcdbe..d755040 100644
> --- a/src/core/socket.c
> +++ b/src/core/socket.c
> @@ -308,7 +308,7 @@ static int socket_add_device_link(Socket *s) {
>  
>          assert(s);
>  
> -        if (!s->bind_to_device)
> +        if (!s->bind_to_device || streq(s->bind_to_device, "lo"))
>                  return 0;
>  
>          if (asprintf(&t, "/sys/subsystem/net/devices/%s", s->bind_to_device) < 0)


Lennart

-- 
Lennart Poettering - Red Hat, Inc.


More information about the systemd-devel mailing list