¡Hola Marcos! El 2016-10-18 a las 20:25 +0200, Marcos Dione escribió:
marble has for ages disabled symlinks traversal in a great part of the local cache code. I know for certain that developers have decided to do so to avoid user reports related to dangling symlinks (sorry, I can't find a reference now).
This also avoids the "recursion too deep" problems.
This bug report is to see if the Debian maintainers agree this is wrong.
What's the benefit that you seek by enabling symlinks? As package maintainers we prefer to avoid diverging too much from upstream.In particular, this patch is simple enough, but it creates a vector for the kind of bugs that upstream wants to avoid. How would we avoid dangling symlinks problems?
Attached you'll find a patch that removes all the references to QDir::NoSymLinks from the code. I tested it and it works fine now. I will keep having this patch around, but I hope you find it useful too, so everyone else can profit from it, and I don't have to recompile marble from time to time :)
How about trying to send this upstream, maybe making this an option? Happy hacking, -- "Las computadoras son inútiles, solo pueden darte respuestas." -- Pablo Picasso Saludos /\/\ /\ >< `/
signature.asc
Description: PGP signature
