Code Review / controller.git / commit
? search:
summary | shortlog | log | commit | commitdiff | review | tree
(parent: 3bc363a) | patch
Fix NPE in AbstractFeatureWrapper 71/27971/1
authorTom Pantelis <tpanteli@brocade.com>
Mon, 5 Oct 2015 16:10:23 +0000 (12:10 -0400)
committerTony Tkacik <ttkacik@cisco.com>
Tue, 6 Oct 2015 15:18:12 +0000 (15:18 +0000)
commitb6d6f433f22283fc5b078c1129bd24ff459fd799
tree0c80b0edc6af2db1f4648f13332d80de9ca8f039tree | snapshot
parent3bc363a69d6d48709f7fd741ef018ecd75b8f99bcommit | diff
Fix NPE in AbstractFeatureWrapper

Added check in ChildAwareFeatureWrapper#getChildFeatures to verify the
feature exists in the FeaturesService to avoid NPE. This is a similar
workaround as was done in FeatureConfigPusher for a bug in karaf where
the FeaturesService may mysteriously return null for an existing feature.

Change-Id: I006cd012e919ac206d70bb4ee5754c72f0f01b32
Signed-off-by: Tom Pantelis <tpanteli@brocade.com>
(cherry picked from commit e6076f69f57fe5918c66637414175c6229635841)
opendaylight/config/config-persister-feature-adapter/src/main/java/org/opendaylight/controller/configpusherfeature/internal/ChildAwareFeatureWrapper.java diff | blob | history
OpenDaylight controller