Menu

#38 setPluginPlaces should not presume it was given a list

Next Minor Release
closed
nobody
None
1
2019-07-27
2018-05-01
Anonymous
No

If accidentally instead of a list of paths one provides a single path to setPluginPlaces the map call in locatePlugins will recursively walk from the first character of the path, typically "/" which is not good.

Discussion

  • Thibauld Nion

    Thibauld Nion - 2018-09-02

    Well, it is expecting a list of enumerable (generator for instance) of paths, so if you provide something else, things go bad indeed, but that's true for any of the methods.

    However I get your point, that providing a single string may be tempting and looking (possibly recursively) below "/" is pretty bad.

    So I'll add a specific protection against this precise case.

     
  • Thibauld Nion

    Thibauld Nion - 2018-09-02
    • status: open --> pending
     
  • Thibauld Nion

    Thibauld Nion - 2019-07-27
    • status: pending --> closed
     
  • Thibauld Nion

    Thibauld Nion - 2019-07-27

    The fix is part of Yapsy1.12

     

Anonymous
Anonymous

Add attachments
Cancel