You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@zookeeper.apache.org by "Mahadev konar (JIRA)" <ji...@apache.org> on 2009/01/08 22:07:00 UTC

[jira] Commented: (ZOOKEEPER-268) tostring on jute generated objects can cause NPE

    [ https://issues.apache.org/jira/browse/ZOOKEEPER-268?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12662117#action_12662117 ] 

Mahadev konar commented on ZOOKEEPER-268:
-----------------------------------------

the patch looks good except a nit.. 

{code}
  /** Cannot create a new instance of Utils */
     private Utils() {
+        super();
     }

{code}

why do we need to call super()? isnt it default object()?

> tostring on jute generated objects can cause NPE
> ------------------------------------------------
>
>                 Key: ZOOKEEPER-268
>                 URL: https://issues.apache.org/jira/browse/ZOOKEEPER-268
>             Project: Zookeeper
>          Issue Type: Bug
>          Components: java client, server
>    Affects Versions: 3.0.0, 3.0.1
>            Reporter: Patrick Hunt
>            Assignee: Patrick Hunt
>             Fix For: 3.1.0
>
>         Attachments: ZOOKEEPER-268.patch
>
>
> Jute still causing problems with tostring operations on generated code, need to review/cleanup the toCSV code
> From user Kevin Burton:
> ---------------------------------------------
> Creating this node with this ACL:
> Created /foo
> setAcl /foo world:anyone:w
> Causes the exception included below.
> It's an infinite loop so it's just called over and over again filling my
> console.
> I'm just doing an exists( path, true ); ... setting a watch still causes the
> problem.
> java.lang.NullPointerException
>         at org.apache.jute.Utils.toCSVBuffer(Utils.java:234)
>         at
> org.apache.jute.CsvOutputArchive.writeBuffer(CsvOutputArchive.java:101)
>         at
> org.apache.zookeeper.proto.GetDataResponse.toString(GetDataResponse.java:48)
>         at java.lang.String.valueOf(String.java:2827)
>         at java.lang.StringBuilder.append(StringBuilder.java:115)
>         at
> org.apache.zookeeper.ClientCnxn$Packet.toString(ClientCnxn.java:230)
>         at java.lang.String.valueOf(String.java:2827)
>         at java.lang.StringBuilder.append(StringBuilder.java:115)
>         at
> org.apache.zookeeper.ClientCnxn$SendThread.readResponse(ClientCnxn.java:586)
>         at
> org.apache.zookeeper.ClientCnxn$SendThread.doIO(ClientCnxn.java:626)
>         at
> org.apache.zookeeper.ClientCnxn$SendThread.run(ClientCnxn.java:852)
> java.lang.NullPointerException
>         at org.apache.jute.Utils.toCSVBuffer(Utils.java:234)
>         at
> org.apache.jute.CsvOutputArchive.writeBuffer(CsvOutputArchive.java:101)
>         at
> org.apache.zookeeper.proto.GetDataResponse.toString(GetDataResponse.java:48)
>         at java.lang.String.valueOf(String.java:2827)
>         at java.lang.StringBuilder.append(StringBuilder.java:115)
>         at
> org.apache.zookeeper.ClientCnxn$Packet.toString(ClientCnxn.java:230)
>         at java.lang.String.valueOf(String.java:2827)
>         at java.lang.StringBuilder.append(StringBuilder.java:115)
>         at
> org.apache.zookeeper.ClientCnxn$SendThread.readResponse(ClientCnxn.java:586)
>         at
> org.apache.zookeeper.ClientCnxn$SendThread.doIO(ClientCnxn.java:626)
>         at
> org.apache.zookeeper.ClientCnxn$SendThread.run(ClientCnxn.java:852)

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.