NamespaceProcessor throws NPE in renewLease().

GiraffaFS / giraffa

Giraffa FileSystem (Slack: giraffa-fs.slack.com)

https://giraffa.ci.cloudbees.com

Apache License 2.0

17 stars 6 forks source link

NamespaceProcessor throws NPE in renewLease(). #136

Closed shvachko closed 9 years ago

shvachko commented 9 years ago

In NamespaceProcessor.renewLease() leaseManager.renewLease() returns null if there are no leases for the client. And then the for-loop throws NPE. Should check not-null or return empty collection.

pjeli commented 9 years ago

Opened pull request. Pretty simple fix.

shvachko commented 9 years ago

+1 LGTM

shvachko commented 9 years ago

Committed. Thank you Plamen.