From e8c92fd5145647fb8ee52462f7b1417f5df94f81 Mon Sep 17 00:00:00 2001 From: Max Kellermann Date: Tue, 29 Sep 2009 20:36:06 +0200 Subject: [PATCH] screen_client: don't update twice in screen_auth() One mpdclient_update() call is in _screen_auth(), and the second is in screen_auth(). screen_auth() does not check whether _screen_auth() was successful prior to that. --- src/screen_client.c | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/src/screen_client.c b/src/screen_client.c index 855a1e2..397a7cb 100644 --- a/src/screen_client.c +++ b/src/screen_client.c @@ -49,7 +49,5 @@ _screen_auth(struct mpdclient *c, gint recursion) gint screen_auth(struct mpdclient *c) { - gint ret = _screen_auth(c, 0); - mpdclient_update(c); - return ret; + return _screen_auth(c, 0); } -- 2.30.2