Hi,
Why not return -1 in spice_server_get_sock_info() and
spice_server_get_peer_info() and remove these main channel functions.
Also the public function should be marked as deprecated if their usage
should be avoided
Pavel
On Fri, 2016-11-25 at 14:52 +0000, Frediano Ziglio wrote:
> These functions are not used since years and are not supporting
> multiple clients.
>
> Signed-off-by: Frediano Ziglio <fziglio@xxxxxxxxxx>
> ---
> server/main-channel.c | 8 ++++----
> 1 file changed, 4 insertions(+), 4 deletions(-)
>
> diff --git a/server/main-channel.c b/server/main-channel.c
> index 37b2f25..3d107c4 100644
> --- a/server/main-channel.c
> +++ b/server/main-channel.c
> @@ -306,16 +306,16 @@ MainChannelClient
> *main_channel_link(MainChannel *channel, RedClient *client,
> return mcc;
> }
>
> +/* very obsolete and old function, retain only for ABI */
> int main_channel_getsockname(MainChannel *main_chan, struct
> sockaddr *sa, socklen_t *salen)
> {
> - return main_chan ?
> - getsockname(red_channel_get_first_socket(RED_CHANNEL(main_c
> han)), sa, salen) : -1;
> + return -1;
> }
>
> +/* very obsolete and old function, retain only for ABI */
> int main_channel_getpeername(MainChannel *main_chan, struct
> sockaddr *sa, socklen_t *salen)
> {
> - return main_chan ?
> - getpeername(red_channel_get_first_socket(RED_CHANNEL(main_c
> han)), sa, salen) : -1;
> + return -1;
> }
>
> MainChannel* main_channel_new(RedsState *reds)
_______________________________________________
Spice-devel mailing list
Spice-devel@xxxxxxxxxxxxxxxxxxxxx
https://lists.freedesktop.org/mailman/listinfo/spice-devel