Spearhead/node-spearhead
Archived
3
0
Fork 0
Code Issues Pull Requests Releases 1 Wiki Activity

Protect against cache lookup returning null

Browse Source 
If the cache file does not exist _cacheGetJson will not pass any value to the callback.

In the getImage function the callback passed to _cacheGetJson assumes if there was no error value then the second argument must be an array.

Add some logic to guard against null values before iterating over the array.
This commit is contained in:
Kris Shannon 2016-02-15 03:58:29 +11:00
parent 6397c8bade
commit 52fc48b3e9
1 changed files with 6 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

10
lib/tritonapi.js
View File

@ -354,10 +354,12 @@ TritonApi.prototype.getImage = function getImage(opts, cb) {
next(err); next(err);
return; return;
} }
for (var i = 0; i < images.length; i++) { if (images) {
if (images[i].id === opts.name) { for (var i = 0; i < images.length; i++) {
img = images[i]; if (images[i].id === opts.name) {
break; img = images[i];
break;
}
} }
} }
next(); next();