[vlc-commits] satip: fix non-sensical logic (fixes #19634)

Rémi Denis-Courmont git at videolan.org
Tue Feb 13 18:05:09 CET 2018


vlc/vlc-3.0 | branch: master | Rémi Denis-Courmont <remi at remlab.net> | Tue Feb 13 19:02:47 2018 +0200| [aeb09e0220dd30387869dd58356cfb935fed7da5] | committer: Rémi Denis-Courmont

satip: fix non-sensical logic (fixes #19634)

(cherry picked from commit 15660bca6c2eb6e3856c7fbb48cf8cb7da275d62)
Signed-off-by: Rémi Denis-Courmont <remi at remlab.net>

> http://git.videolan.org/gitweb.cgi/vlc/vlc-3.0.git/?a=commit;h=aeb09e0220dd30387869dd58356cfb935fed7da5
---

 modules/access/satip.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/modules/access/satip.c b/modules/access/satip.c
index 2da403dbf8..2e173cfb81 100644
--- a/modules/access/satip.c
+++ b/modules/access/satip.c
@@ -410,7 +410,7 @@ static void satip_teardown(void *data) {
 
             /* Some SATIP servers send a few empty extra bytes after TEARDOWN.
              * Try to read them, to avoid a TCP socket reset */
-            while ((len = recv(sys->tcp_sock, discard_buf, sizeof(discard_buf), 0) > 0));
+            while (recv(sys->tcp_sock, discard_buf, sizeof(discard_buf), 0) > 0);
 
             /* Extra sleep for compatibility with some satip servers, that
              * can't handle new sessions right after teardown */



More information about the vlc-commits mailing list