khammond    01/11/23 08:09:15

  Modified:    src/org/apache/jmeter/save/handlers FilerHandler.java
  Log:
  Patch for bug #5037 submitted by James Boutcher.
  
  Revision  Changes    Path
  1.5       +35 -17    
jakarta-jmeter/src/org/apache/jmeter/save/handlers/FilerHandler.java
  
  Index: FilerHandler.java
  ===================================================================
  RCS file: 
/home/cvs/jakarta-jmeter/src/org/apache/jmeter/save/handlers/FilerHandler.java,v
  retrieving revision 1.4
  retrieving revision 1.5
  diff -u -r1.4 -r1.5
  --- FilerHandler.java 2001/08/16 18:17:56     1.4
  +++ FilerHandler.java 2001/11/23 16:09:15     1.5
  @@ -25,22 +25,40 @@
   
        public void save(Saveable saveObject, Writer out) throws java.io.IOException
        {
  -             Filer save = (Filer)saveObject;
  -             out.write("<Filer type=\"");
  -             out.write(JMeterHandler.convertToXML(save.getClass().getName()));
  -             out.write("\" name=\"");
  -             out.write(JMeterHandler.convertToXML(save.getName()));
  -             out.write("\" verbose=\"");
  -             out.write(""+save.getVerbose());
  -             out.write("\" append=\"");
  -             out.write(""+save.getAppend());
  -                out.write("\" autoFlush=\"");
  -                out.write("" + save.getAutoFlush());
  -                out.write("\" viewSubmitData=\"");
  -                out.write("" + save.getViewSubmitData());
  -             out.write("\" file=\"");
  -             out.write(JMeterHandler.convertToXML(save.getFile()));
  -             out.write("\"/>");
  +             // protect ourselves from bad input.
  +             if ((saveObject == null) || (out == null))
  +             {
  +                     // error!
  +             }
  +             else
  +             {
  +                     Filer save = (Filer)saveObject;
  +
  +                     // validate filename
  +                     String filename = save.getFile();
  +                     if (filename == null)
  +                     {
  +                             System.err.println("Error. Missing filename in File 
Reporter.");
  +                     }
  +                     else
  +                     {
  +                             out.write("<Filer type=\"");
  +                             
out.write(JMeterHandler.convertToXML(save.getClass().getName()));
  +                             out.write("\" name=\"");
  +                             out.write(JMeterHandler.convertToXML(save.getName()));
  +                             out.write("\" verbose=\"");
  +                             out.write(""+save.getVerbose());
  +                             out.write("\" append=\"");
  +                             out.write(""+save.getAppend());
  +                             out.write("\" autoFlush=\"");
  +                             out.write("" + save.getAutoFlush());
  +                             out.write("\" viewSubmitData=\"");
  +                             out.write("" + save.getViewSubmitData());
  +                             out.write("\" file=\"");
  +                             out.write(JMeterHandler.convertToXML(filename));
  +                             out.write("\"/>");
  +                     }
  +             }
        }
   
        public void setAtts(Attributes atts) throws java.lang.Exception
  @@ -70,4 +88,4 @@
        {
                return model;
        }
  -}
  \ No newline at end of file
  +}
  
  
  

--
To unsubscribe, e-mail:   <mailto:[EMAIL PROTECTED]>
For additional commands, e-mail: <mailto:[EMAIL PROTECTED]>

Reply via email to