From d353fca7db81e29639261310d95abfafedbcd138 Mon Sep 17 00:00:00 2001 From: Greg Hogan Date: Fri, 29 Jul 2016 14:54:50 -0700 Subject: [PATCH] Android: Fix NPE in configuring brush Fixing a NPE found when running the Example test, since the brush returned from getDefinedBrush() may be null. --- .../src/main/java/com/horcrux/svg/RNSVGPathShadowNode.java | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/android/src/main/java/com/horcrux/svg/RNSVGPathShadowNode.java b/android/src/main/java/com/horcrux/svg/RNSVGPathShadowNode.java index 50dbaa16..9bd1126e 100644 --- a/android/src/main/java/com/horcrux/svg/RNSVGPathShadowNode.java +++ b/android/src/main/java/com/horcrux/svg/RNSVGPathShadowNode.java @@ -310,7 +310,9 @@ public class RNSVGPathShadowNode extends RNSVGVirtualNode { mPath.computeBounds(box, true); } PropHelper.RNSVGBrush brush = getSvgShadowNode().getDefinedBrush(colors.getString(1)); - brush.setupPaint(paint, box, mScale, opacity); + if (brush != null) { + brush.setupPaint(paint, box, mScale, opacity); + } } else { // TODO: Support pattern. FLog.w(ReactConstants.TAG, "RNSVG: Color type " + colorType + " not supported!");