On Fri, Oct 05, 2012 at 06:22:25PM +0200, Michael Hanselmann wrote: > 2012/10/5 Iustin Pop <[email protected]>: > > On Fri, Oct 05, 2012 at 03:55:39AM +0200, Michael Hanselmann wrote: > >> +def CheckFileStoragePath(path, > >> _filename=pathutils.FILE_STORAGE_PATHS_FILE): > >> + allowed = LoadAllowedFileStoragePaths(_filename) > >> + > >> + return _CheckFileStoragePath(path, allowed) > > > > When reading the patch, this return was confusing, since _CheckFile… > > doesn't have an explicit return value. > > That was indeed not what intended. Changed it to > “_CheckFileStoragePath(path, LoadAllowedFileStoragePaths(_filename))”.
LGTM, thanks. iustin
