<div id="geary-body" dir="auto"><div>On Fri, Mar 19, 2021 at 14:44, Sean McGovern <gseanmcg@gmail.com> wrote:<br><blockquote type="cite"><div class="plaintext" style="white-space: pre-wrap;">I just had a "devil's advocate" kind of thought go through my head
when reading through this patchset.

Should the ML component possibly have security restrictions to prevent
other modules in VLC from querying it when really they shouldn't?</div></blockquote><span style="white-space: pre-wrap;"><br></span></div><div><span style="white-space: pre-wrap;">Hi,</span></div><div><span style="white-space: pre-wrap;">There's no such protection for the moment. I actually thought the medialib instance</span></div><div><span style="white-space: pre-wrap;">was only queryable by an interface module like the main playlist but its currently not</span></div><div><span style="white-space: pre-wrap;">the case.</span></div><div><span style="white-space: pre-wrap;">But in the case of the main playlist for example, this is only a matter of guiding the</span></div><div><span style="white-space: pre-wrap;">modules implementers and tell them that they shouldn't access the main playlist</span></div><div><span style="white-space: pre-wrap;">if they are not working on an interface module.</span></div><div><span style="white-space: pre-wrap;">Truly denying modules access to </span><span style="white-space: pre-wrap;">certains </span><span style="white-space: pre-wrap;">parts of the core would require isolating</span></div><div><span style="white-space: pre-wrap;">modules in separate processes </span><span style="white-space: pre-wrap;">and that's not where we are now.</span></div><div><span style="white-space: pre-wrap;"><br></span></div><div>On Sat, Mar 20, 2021 at 23:58, Jean-Baptiste Kempf <jb@videolan.org> wrote:<br><blockquote type="cite"><div class="plaintext" style="white-space: pre-wrap;">Please mention 
Close #261 in the commit log.
</div></blockquote><span style="white-space: pre-wrap;"><br>Sure, I added `Implements #261` at the end of the commit message as It has</span></div><div><span style="white-space: pre-wrap;">the same behavior and will close the issue once merged!</span></div><div><span style="white-space: pre-wrap;"><br></span></div><div><span style="white-space: pre-wrap;">Regards</span></div></div>