<html><head></head><body>On second consideration, chopping the path is not correct behaviour. This patch is concealing the bug highlighted by the warning rather than fixing it IMO.<br><br>Better drop the feature altogether as I dot think anyone's using it.<br><br><div class="gmail_quote">Le 20 janvier 2020 09:38:46 GMT+02:00, "Rémi Denis-Courmont" <remi@remlab.net> a écrit :<blockquote class="gmail_quote" style="margin: 0pt 0pt 0pt 0.8ex; border-left: 1px solid rgb(204, 204, 204); padding-left: 1ex;">
OK, though I'd suggest removing Unix socket support entirely as the old RC "protocol" is not suitable for programmatic usage.<br><br><div class="gmail_quote">Le 19 janvier 2020 17:39:27 GMT+02:00, Alexandre Janniaux <ajanni@videolabs.io> a écrit :<blockquote class="gmail_quote" style="margin: 0pt 0pt 0pt 0.8ex; border-left: 1px solid rgb(204, 204, 204); padding-left: 1ex;">
<pre class="k9mail">Fortify enabled implies a warning if we use strncpy with a length equal<br>to the buffer size. As we fill the last item with a null character, we<br>can use length-1 instead as well.<hr> modules/control/rc.c | 2 +-<br> 1 file changed, 1 insertion(+), 1 deletion(-)<br><br>diff --git a/modules/control/rc.c b/modules/control/rc.c<br>index cd85602d59..dc37d352ad 100644<br>--- a/modules/control/rc.c<br>+++ b/modules/control/rc.c<br>@@ -1730,7 +1730,7 @@ static int Activate( vlc_object_t *p_this )<br>         }<br> <br>         addr.sun_family = AF_LOCAL;<br>-        strncpy( addr.sun_path, psz_unix_path, sizeof( addr.sun_path ) );<br>+        strncpy( addr.sun_path, psz_unix_path, sizeof( addr.sun_path ) - 1 );<br>         addr.sun_path[sizeof( addr.sun_path ) - 1] = '\0';<br> <br>         if (bind (i_socket, (struct sockaddr *)&addr, sizeof (addr))</pre></blockquote></div></blockquote></div><br>-- <br>Envoyé de mon appareil Android avec Courriel K-9 Mail. Veuillez excuser ma brièveté.</body></html>