Submitting patches for problems found by static analysis

Bill Pugh pugh at cs.umd.edu
Mon Jun 18 19:23:15 PDT 2007


Does anyone have suggestions for how to go about submitting patches  
for issues found via static analysis?

One of the problems is that static analysis just points me at the  
source file, and it can often be exceptionally difficult for me as an  
outsider to figure out who the responsible engineer is, or if an  
appropriate OpenJDK project exists.

For example, below is some code from FaultReason with an obvious bug:  
the line assert(texts == null) should be assert(texts != null). But I  
have no idea who to contact about fixing this bug.

The code in NamingContextImpl below also has an obvious bug. The  
check (name != null || name.length > 0) should be (name != null &&  
name.length > 0); actually, just (name != null) is probably even better.

I'd like to submit about 100 patches, so making this process as easy  
as possible is important.

Bill

--------------------------
package com.sun.xml.internal.ws.encoding.soap.message;
....

/**
  * SOAP 1.2 Fault Reason
  *  <soapenv:Reason>
  *      <soapenv:Text xml:lang="en">...</soapenv:Text>
  *  </soapenv:Reason>
  *
  * @author Vivek Pandey
  */
public class FaultReason {
     private List<FaultReasonText> texts;

     public FaultReason(FaultReasonText... texts) {
         assert(texts == null);
         this.texts = Arrays.asList(texts);
     }

--------------------------
package com.sun.corba.se.impl.naming.pcosnaming;

...

public class NamingContextImpl
     extends NamingContextExtPOA
     implements NamingContextDataStore, Serializable
{

...
    public static String nameToString(NameComponent[] name)
     {
         StringBuffer s = new StringBuffer("{");
         if (name != null || name.length > 0) {
           for (int i=0;i<name.length;i++) {



More information about the quality-discuss mailing list