Re: Updated unveil patch

Le 28/07/2024 à 19:14, [email protected] a écrit :
> Hi,
> 
> Here is a new patch. I have done quite a bit more work on this and
> think it may be close to completion.
> 
> The '~/Downloads' directory has been unveiled to match the behavior of
> Firefox and Chromium on OpenBSD, but Dillo's default of '/tmp'
> continues to work as well.
> 
> I have also made sure everything works fine when there is no ~/.dillo
> directory, Dillo can create it, and also can access the system defaults
> in '/usr/local/etc/dillo'.
>   
> dpid is also now unveiled, as well as all of the stock plugins except
> hello.dpi, I didn't see any point to that.
> 
> Here are some other tests which I have run:
> 
> - Regular browsing works fine
> - Connect to an FTP site and download a file, also view a text file and
>    view an image
> - Open a text and image file from /tmp and ~/Downloads
> - Add/remove bookmarks
> - Download a file to /tmp and ~/Downloads
> - Save a page to /tmp and ~/Downloads
> - View source still works
> - Fonts and cursor icons are working correctly
> - data: uri works correctly with text and images
> 
> So far everything seems to be fine. I will keep testing, but would
> really appreciate some help with reviewing this, there could be some
> edge-cases which I missed.
> 
> Regards,
> Alex
> 
> 

you should provide a command line argument to disable sandboxing, so in case
of a problem users can run dillo without sandboxing and see if it works better,
allowing to figure if sandboxing is the root cause of their problem.

what happens if you have no ~/.dillo when you start dillo with unveil?

why do you need to unveil the same directories multiple times in the code?
_______________________________________________
Dillo-dev mailing list -- [email protected]
To unsubscribe send an email to [email protected]