client: combine client_open() and client_parse_config()
authorAndre Noll <maan@systemlinux.org>
Sun, 4 Feb 2007 17:55:12 +0000 (18:55 +0100)
committerAndre Noll <maan@systemlinux.org>
Sun, 4 Feb 2007 17:55:12 +0000 (18:55 +0100)
commit1b46ae4193cddb7d589b0f2f7d8158b84f7e9f6d
treeb73469079e4045185e950ad53813b3d2f6268a19
parent32aeef08d1132ef9789ca16654ed1ec2b4dbbc42
client: combine client_open() and client_parse_config()

Both users (client.c and audiod.c) called client_open() right
after calling client_parse_config(), so call client_open() from
client_parse_config() and make client_open() static.

While we're at it, rename client_open() to client_connect() as this
is what the function does. Also rename client_parse_config() to
client_open() and change audiod.c and client.c to call client_open()
instead of client_parse_config().

This patch fixes a segfault in para_client was it was run without
specifying a command. In this case, the private client data struct
was used by para_log after being freed -- Ouch. Fortunately, this
bug didn't affect para_audiod as it never calls these functions with
no command.

Another bug due to a typo in client_close() is also fixed. This code
needs closer investigation and lacks documentation btw.
audiod.c
client.c
client.h
client_common.c