* [#14839](http://bugzilla.scilab.org/show_bug.cgi?id=14839): `plot2d2` crashed Scilab. 51/18651/2
Adeline CARNIS [Mon, 7 Nov 2016 11:54:43 +0000 (12:54 +0100)]
Change-Id: I37adea67bea4da901b2babf0a7da8fd9413dce2b

scilab/CHANGES.md
scilab/modules/graphics/sci_gateway/c/sci_plot2d1.c
scilab/modules/graphics/tests/nonreg_tests/bug_14839.tst [new file with mode: 0644]

index bb457ab..e9053d2 100644 (file)
@@ -414,6 +414,7 @@ Bug Fixes
 * [#14775](http://bugzilla.scilab.org/show_bug.cgi?id=14775): Loading empty (0 bytes) .sod File crashed scilab
 * [#14808](http://bugzilla.scilab.org/show_bug.cgi?id=14808): E=[ 'A' 'B' 'C' 'D' 'E']  ,  E(0:0) Crash Scilab Console
 * [#14821](http://bugzilla.scilab.org/show_bug.cgi?id=14821): `getio` function was missing. An error on the diary file opened has been corrected
+* [#14839](http://bugzilla.scilab.org/show_bug.cgi?id=14839): `plot2d2` crashed Scilab.
 * [#14887](http://bugzilla.scilab.org/show_bug.cgi?id=14887): For many graphic handle types, the display of the `.tag` value missed `".."` delimiters
 
 ### Bugs fixed in 6.0.0 beta-2 and earlier 6.0.0 pre-releases:
index 128b5dc..4ea837b 100644 (file)
@@ -323,7 +323,7 @@ int sci_plot2d1_G(char * fname, int ptype, void *pvApiCtx)
         FREE(style);
         return 0;
     }
-    freeStrf = isDefStrf(strf);
+    freeStrf = !(isDefStrf(strf));
     if (get_legend_arg(pvApiCtx, fname, 5 + iskip, opts, &legend) == 0)
     {
         if (freeStrf)
@@ -333,7 +333,7 @@ int sci_plot2d1_G(char * fname, int ptype, void *pvApiCtx)
         FREE(style);
         return 0;
     }
-    freeLegend = isDefLegend(legend);
+    freeLegend = !(isDefLegend(legend));
     if (get_rect_arg(pvApiCtx, fname, 6 + iskip, opts, &rect) == 0)
     {
         if (freeLegend)
diff --git a/scilab/modules/graphics/tests/nonreg_tests/bug_14839.tst b/scilab/modules/graphics/tests/nonreg_tests/bug_14839.tst
new file mode 100644 (file)
index 0000000..421c1cd
--- /dev/null
@@ -0,0 +1,21 @@
+// =============================================================================
+// Scilab ( http://www.scilab.org/ ) - This file is part of Scilab
+// Copyright (C) 2016 - Scilab Enterprises - Adeline CARNIS
+//
+//  This file is distributed under the same license as the Scilab package.
+// =============================================================================
+
+// <-- TEST WITH GRAPHIC -->
+
+// <-- INTERACTIVE TEST -->
+
+// <-- Non-regression test for bug 14839 -->
+//
+// <-- Bugzilla URL -->
+// http://bugzilla.scilab.org/14839
+//
+// <-- Short Description -->
+// plot2d2 crashed Scilab.
+
+t=linspace(0,10,61);
+plot2d2(t,t);