ofbiz-notifications mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Michael Brohl (JIRA)" <j...@apache.org>
Subject [jira] [Closed] (OFBIZ-9715) [FB] Package org.apache.ofbiz.datafile
Date Sat, 28 Oct 2017 14:46:00 GMT

     [ https://issues.apache.org/jira/browse/OFBIZ-9715?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]

Michael Brohl closed OFBIZ-9715.
--------------------------------
       Resolution: Implemented
    Fix Version/s: Upcoming Release

Thanks Julian,

your modified patch is in trunk r1813637.


> [FB] Package org.apache.ofbiz.datafile
> --------------------------------------
>
>                 Key: OFBIZ-9715
>                 URL: https://issues.apache.org/jira/browse/OFBIZ-9715
>             Project: OFBiz
>          Issue Type: Sub-task
>          Components: ALL APPLICATIONS, ALL COMPONENTS
>    Affects Versions: Trunk
>            Reporter: Julian Leichert
>            Assignee: Michael Brohl
>            Priority: Minor
>             Fix For: Upcoming Release
>
>         Attachments: OFBIZ-9715_org.apache.ofbiz.datafile_bugfixes.patch
>
>
> DataFile.java:76, RCN_REDUNDANT_NULLCHECK_OF_NONNULL_VALUE
> - RCN: Redundant nullcheck of reader, which is known to be non-null in org.apache.ofbiz.datafile.DataFile.makeDataFile(URL,
String)
> This method contains a redundant check of a known non-null value against the constant
null.
> DataFile.java:139, DM_DEFAULT_ENCODING
> - Dm: Found reliance on default encoding in org.apache.ofbiz.datafile.DataFile.readDataFile(String):
String.getBytes()
> 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.
> DataFile.java:190, RCN_REDUNDANT_NULLCHECK_OF_NONNULL_VALUE
> - RCN: Redundant nullcheck of fos, which is known to be non-null in org.apache.ofbiz.datafile.DataFile.writeDataFile(String)
> This method contains a redundant check of a known non-null value against the constant
null.
> DataFile.java:206, DM_DEFAULT_ENCODING
> - Dm: Found reliance on default encoding in org.apache.ofbiz.datafile.DataFile.writeDataFile():
java.io.ByteArrayOutputStream.toString()
> 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.
> DataFile.java:209, RCN_REDUNDANT_NULLCHECK_OF_NONNULL_VALUE
> - RCN: Redundant nullcheck of bos, which is known to be non-null in org.apache.ofbiz.datafile.DataFile.writeDataFile()
> This method contains a redundant check of a known non-null value against the constant
null.
> DataFile.java:230, DM_DEFAULT_ENCODING
> - Dm: Found reliance on default encoding in org.apache.ofbiz.datafile.DataFile.writeRecords(OutputStream,
List): String.getBytes()
> 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.
> DataFile2EntityXml.java:52, OS_OPEN_STREAM_EXCEPTION_PATH
> - OS: org.apache.ofbiz.datafile.DataFile2EntityXml.writeToEntityXml(String, DataFile)
may fail to close stream on exception
> The method creates an IO stream object, does not assign it to any fields, pass it to
other methods, or return it, and does not appear to close it on all possible exception paths
out of the method.  This may result in a file descriptor leak.  It is generally a good idea
to use a finally block to ensure that streams are closed.
> DataFile2EntityXml.java:102, DM_DEFAULT_ENCODING
> - Dm: Found reliance on default encoding in org.apache.ofbiz.datafile.DataFile2EntityXml.main(String[]):
new java.io.FileWriter(String)
> 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.
> DataFile2EntityXml.java:102, OS_OPEN_STREAM_EXCEPTION_PATH
> - OS: org.apache.ofbiz.datafile.DataFile2EntityXml.main(String[]) may fail to close stream
on exception
> The method creates an IO stream object, does not assign it to any fields, pass it to
other methods, or return it, and does not appear to close it on all possible exception paths
out of the method.  This may result in a file descriptor leak.  It is generally a good idea
to use a finally block to ensure that streams are closed.
> DataFile2EntityXml.java:121, DE_MIGHT_IGNORE
> - DE: org.apache.ofbiz.datafile.DataFile2EntityXml.main(String[]) might ignore java.lang.Exception
> This method might ignore an exception.  In general, exceptions should be handled or reported
in some way, or they should be thrown out of the method.
> DataFile2EntityXml.java:128, NP_NULL_ON_SOME_PATH
> - NP: Possible null pointer dereference of dataFile in org.apache.ofbiz.datafile.DataFile2EntityXml.main(String[])
> There is a branch of statement that, if executed, guarantees that a null value will be
dereferenced, which would generate a NullPointerException when the code is executed. Of course,
the problem might be that the branch or statement is infeasible and that the null pointer
exception can't ever be executed; deciding that is beyond the ability of FindBugs.
> ModelDataFileReader.java:99, RCN_REDUNDANT_NULLCHECK_OF_NONNULL_VALUE
> - RCN: Redundant nullcheck of modelRecord, which is known to be non-null in org.apache.ofbiz.datafile.ModelDataFileReader.createModelDataFile(Element)
> This method contains a redundant check of a known non-null value against the constant
null.
> ModelDataFileReader.java:152, RCN_REDUNDANT_NULLCHECK_OF_NONNULL_VALUE, Priorit├Ąt: Normal
> - RCN: Redundant nullcheck of dataFile, which is known to be non-null in org.apache.ofbiz.datafile.ModelDataFileReader.createModelDataFiles()
> This method contains a redundant check of a known non-null value against the constant
null.
> ModelField.java:27, SE_NO_SERIALVERSIONID
> - SnVI: org.apache.ofbiz.datafile.ModelField is Serializable; consider declaring a serialVersionUID
> This class implements the Serializable interface, but does not define a serialVersionUID
field.  A change as simple as adding a reference to a .class object will add synthetic fields
to the class, which will unfortunately change the implicit serialVersionUID (e.g., adding
a reference to String.class will generate a static field class$java$lang$String). Also, different
source code to bytecode compilers may use different naming conventions for synthetic variables
generated for references to class objects or inner classes. To ensure interoperability of
Serializable across versions, consider adding an explicit serialVersionUID.
> ModelRecord.java:65, URF_UNREAD_PUBLIC_OR_PROTECTED_FIELD
> - UrF: Unread public/protected field: org.apache.ofbiz.datafile.ModelRecord.limit
> This field is never read.  The field is public or protected, so perhaps it is intended
to be used with classes not seen as part of the analysis. If not, consider removing it from
the class.
> Record.java:-1, SE_TRANSIENT_FIELD_NOT_RESTORED
> - Se: The field org.apache.ofbiz.datafile.Record.modelRecord is transient but isn't set
by deserialization
> This class contains a field that is updated at multiple places in the class, thus it
seems to be part of the state of the class. However, since the field is marked as transient
and not set in readObject or readResolve, it will contain the default value in any deserialized
instance of the class.
> Record.java:-1, UWF_NULL_FIELD, Priorit├Ąt: Niedrig
> - UwF: Field only ever set to null: org.apache.ofbiz.datafile.Record.parentRecord
> All writes to this field are of the constant value null, and thus all reads of the field
will return null. Check for errors, or remove it if it is useless.
> Record.java:52, SE_NO_SERIALVERSIONID
> - SnVI: org.apache.ofbiz.datafile.Record is Serializable; consider declaring a serialVersionUID
> This class implements the Serializable interface, but does not define a serialVersionUID
field.  A change as simple as adding a reference to a .class object will add synthetic fields
to the class, which will unfortunately change the implicit serialVersionUID (e.g., adding
a reference to String.class will generate a static field class$java$lang$String). Also, different
source code to bytecode compilers may use different naming conventions for synthetic variables
generated for references to class objects or inner classes. To ensure interoperability of
Serializable across versions, consider adding an explicit serialVersionUID.
> Record.java:85, UG_SYNC_SET_UNSYNC_GET
> - UG: org.apache.ofbiz.datafile.Record.get(String) is unsynchronized, org.apache.ofbiz.datafile.Record.set(String,
Object, boolean) is synchronized
> This class contains similarly-named get and set methods where the set method is synchronized
and the get method is not.  This may result in incorrect behavior at runtime, as callers of
the get method will not necessarily see a consistent state for the object.  The get method
should be made synchronized.
> Record.java:272, DM_DEFAULT_ENCODING
> - Dm: Found reliance on default encoding in org.apache.ofbiz.datafile.Record.setString(String,
String): String.getBytes()
> 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.
> Record.java:307, RCN_REDUNDANT_NULLCHECK_OF_NONNULL_VALUE
> - RCN: Redundant nullcheck of org.apache.ofbiz.datafile.Record.getModelRecord(), which
is known to be non-null in org.apache.ofbiz.datafile.Record.getFixedString(String)
> This method contains a redundant check of a known non-null value against the constant
null.
> RecordIterator.java:233, RCN_REDUNDANT_NULLCHECK_OF_NONNULL_VALUE
> - RCN: Redundant nullcheck of typeCode, which is known to be non-null in org.apache.ofbiz.datafile.RecordIterator.findModelForLine(String,
int, ModelDataFile)
> This method contains a redundant check of a known non-null value against the constant
null.



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)

Mime
View raw message