Bug 13503 fixed: Segfault with xarrows 15/14815/2
Calixte DENIZET [Mon, 7 Jul 2014 12:57:55 +0000 (14:57 +0200)]
Change-Id: I3e807ed415fa2a16eb3b9925afd02e65579fc8a5

scilab/CHANGES_5.5.X
scilab/modules/graphics/sci_gateway/c/sci_xarrows.c
scilab/modules/graphics/tests/nonreg_tests/bug_13503.dia.ref [new file with mode: 0644]
scilab/modules/graphics/tests/nonreg_tests/bug_13503.tst [new file with mode: 0644]

index e91c6a1..47cb80d 100644 (file)
@@ -142,6 +142,8 @@ Scilab Bug Fixes
 
 * Bug #13501 fixed - Typos fixed in English help pages.
 
+* Bug #13503 fixed - xarrows made Scilab crash in some cases.
+
 
 Xcos Bug Fixes
 ==============
index 15b5dd6..af72436 100644 (file)
@@ -139,9 +139,10 @@ int sci_xarrows(char *fname, unsigned long fname_len)
                     Scierror(999, _("%s: Wrong size for input argument #%d: A real scalar expected.\n"), fname, 4);
                     return 1;
                 }
+
+                arsize = *dl4;
+                stylePos = 5;
             }
-            arsize = *dl4;
-            stylePos = 5;
         }
         else if (m3 == 1 && n3 == 1)
         {
diff --git a/scilab/modules/graphics/tests/nonreg_tests/bug_13503.dia.ref b/scilab/modules/graphics/tests/nonreg_tests/bug_13503.dia.ref
new file mode 100644 (file)
index 0000000..3302ff6
--- /dev/null
@@ -0,0 +1,15 @@
+// =============================================================================
+// Scilab ( http://www.scilab.org/ ) - This file is part of Scilab
+// Copyright (C) 2014 - Scilab Enterprises - Calixte DENIZET
+//
+//  This file is distributed under the same license as the Scilab package.
+// =============================================================================
+// <-- TEST WITH GRAPHIC -->
+// <-- Non-regression test for bug 13503 -->
+//
+// <-- Bugzilla URL -->
+// http://bugzilla.scilab.org/13503
+//
+// <-- Short Description -->
+// Segfault with xarrows
+xarrows([0 ; 1], [0 ; 0], [0 ; 0]);
diff --git a/scilab/modules/graphics/tests/nonreg_tests/bug_13503.tst b/scilab/modules/graphics/tests/nonreg_tests/bug_13503.tst
new file mode 100644 (file)
index 0000000..89be5fe
--- /dev/null
@@ -0,0 +1,18 @@
+// =============================================================================
+// Scilab ( http://www.scilab.org/ ) - This file is part of Scilab
+// Copyright (C) 2014 - Scilab Enterprises - Calixte DENIZET
+//
+//  This file is distributed under the same license as the Scilab package.
+// =============================================================================
+
+// <-- TEST WITH GRAPHIC -->
+
+// <-- Non-regression test for bug 13503 -->
+//
+// <-- Bugzilla URL -->
+// http://bugzilla.scilab.org/13503
+//
+// <-- Short Description -->
+// Segfault with xarrows
+
+xarrows([0 ; 1], [0 ; 0], [0 ; 0]);
\ No newline at end of file