Handle special case where path consists of a single move command

- prevents crash, fixes #223
svg-net · Feb 3, 2019 · f7c5fb0 · f7c5fb0
1 parent c409d2e
commit f7c5fb0
Show file tree

Hide file tree

Showing 4 changed files with 17 additions and 1 deletion.
diff --git a/Source/Paths/SvgPath.cs b/Source/Paths/SvgPath.cs
@@ -45,13 +45,27 @@ public override GraphicsPath Path(ISvgRenderer renderer)
         {
             if (this._path == null || this.IsPathDirty)
             {
-                _path = new GraphicsPath();
+                this._path = new GraphicsPath();
 
                 foreach (SvgPathSegment segment in this.PathData)
                 {
                     segment.AddToPath(_path);
                 }
 
+                if (_path.PointCount == 0)
+                {
+                    if (this.PathData.Count > 0)
+                    {
+                        // special case with one move command only, see #223
+                        var segment = this.PathData.Last;
+                        _path.AddLine(segment.End, segment.End);
+                        this.Fill = SvgPaintServer.None;
+                    }
+                    else
+                    {
+                        _path = null;
+                    }
+                }
                 this.IsPathDirty = false;
             }
             return _path;

diff --git a/Tests/Svg.UnitTests/PassingTests.csv b/Tests/Svg.UnitTests/PassingTests.csv
@@ -181,6 +181,8 @@ __issue-191-01
 __issue-202-01
 __issue-214-01
 __issue-215-01
+__issue-223-01
+__issue-223-02
 __issue-227-01
 __issue-227-02
 __issue-239-01

diff --git a/Tests/W3CTestSuite/png/__issue-223-01.png b/Tests/W3CTestSuite/png/__issue-223-01.png
diff --git a/Tests/W3CTestSuite/png/__issue-223-02.png b/Tests/W3CTestSuite/png/__issue-223-02.png