Skip to content
This repository has been archived by the owner on Jan 14, 2018. It is now read-only.

fixed NPE when cache folder is empty #352

Closed
wants to merge 1 commit into from
Closed

fixed NPE when cache folder is empty #352

wants to merge 1 commit into from

Conversation

skhanzada
Copy link

InFileObjectPersister throws NPE from line 112

    List<Object> result = new ArrayList<Object>(cacheFileNameList.length);

The problem is with the 'cacheFileNameList' array that is returned from the following line

String[] cacheFileNameList = getCacheFolder().list(new FilenameFilter() {

if this array is null which could be true in some situations than a NPE is thrown on line 112.

It can be reproduced by clearing app cache from settings and try to use the app, you will see app crash.

@jtietema
Copy link

jtietema commented Sep 2, 2014

@stephanenicolas Any change this can be merged? The failed build seems pretty random, something about Maven not being able to download the findbugs dependency. Doesn't seem related to the PR...

@stephanenicolas
Copy link
Owner

Hi @jtietema , sorry but the patch completely escaped my scope. We thought to go for a 1.5.0 release but it looks like we had less time than we thought for this release yet. Personally, I am completely in something else for now.

So, the solution might be to refork from the last stable version a version 1.4.14, including the patch. I will see what I can do in the next few days. I hope this is fine for you. Thanks to @jtietema and @skhanzada for submitting the patch.

Stéphane

stephanenicolas added a commit that referenced this pull request Sep 6, 2014
@stephanenicolas
Copy link
Owner

Thx. This has been merged. 1.4.14 should be released within hours.

@stephanenicolas
Copy link
Owner

RS 1.4.14 is released and should hit central within 2 hours. Thx for your contributions.

@jtietema
Copy link

Awesome! Thanks for the release :-)

mcesar pushed a commit to project-draco-hr/robospice that referenced this pull request Oct 2, 2016
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants