<!DOCTYPE html><html><head><title></title><style type="text/css">p.MsoNormal,p.MsoNoSpacing{margin:0}</style></head><body><div>Merged, thanks !<br></div><div><br></div><div>On Thu, Nov 19, 2020, at 17:59, Alaric Senat wrote:<br></div><blockquote type="cite" id="qt" style=""><div id="qt-geary-body" dir="auto"><div><br></div><div>The url appended here is the ContrentDirectoryService's control point url of an upnp server. <br></div><div><div>This url is assembled upper in the code so it's probably safe to assume that it will looks like this:<br></div><div>`<a href="http://upnp-server-ip/somepath/ContentDirectoryService">http://upnp-server-ip/somepath/ContentDirectoryService</a>'<br></div><div>Which may be why the patched code was appending the url like that in the first place.<br></div><div><br></div><div>There are some cases where this could fail indeed but since the last part of the url is basically given by the upnp server description, these edges cases could come from a bad server configuration.<br></div></div><div><br></div><div>Alaric<br></div></div><div id="qt-geary-quote" dir="auto"><div>On Thu, Nov 19, 2020 at 17:13, Rémi Denis-Courmont <remi@remlab.net> wrote:<br></div><blockquote type="cite"><div class="qt-plaintext" style="white-space:pre-wrap;">Are you sure that the URL ends with either a list of parameters or a path?
There are other cases that this leaves broken. Blindly appending to a URL is
not a great idea.<br></div></blockquote></div><div>_______________________________________________<br></div><div>vlc-devel mailing list<br></div><div>To unsubscribe or modify your subscription options:<br></div><div><a href="https://mailman.videolan.org/listinfo/vlc-devel">https://mailman.videolan.org/listinfo/vlc-devel</a><br></div></blockquote></body></html>