CoverityFix CID:1350832 Dubious method used 85/20385/3
Nimish Kapoor [Mon, 30 Jul 2018 17:41:50 +0000 (22:41 +0530)]
Found a call to a method which will perform a byte to String (or String to
byte) conversion, and will assume that the default platform encoding is
suitable. This will cause the application behaviour to vary between platforms.
Use an alternative API and specify a charset name or Charset object explicitly.

Change-Id: I2cbaf734f947bfb839d96ec3997c71363cfc754f

scilab/modules/commons/src/java/org/scilab/modules/commons/xml/XConfiguration.java

index 9a4e0db..05f3c45 100644 (file)
@@ -22,6 +22,7 @@ import java.io.File;
 import java.io.FilenameFilter;
 import java.io.FileFilter;
 import java.io.IOException;
+import java.io.UnsupportedEncodingException;
 import java.lang.annotation.Annotation;
 import java.lang.annotation.Retention;
 import java.lang.annotation.RetentionPolicy;
@@ -229,10 +230,12 @@ public class XConfiguration {
         String str = "";
         try {
             transformer.transform(source, result);
-            str = stream.toString();
+            str = stream.toString("UTF-8");
         } catch (TransformerException e) {
             System.err.println("Cannot dump xml");
             return str;
+        } catch (UnsupportedEncodingException e) {
+            e.printStackTrace();
         } finally {
             try {
                 stream.close();