[jira] Commented: (ZOOKEEPER-547) Sanity check in QuorumCnxn Manager and quorum communication port.

["Mahadev konar (JIRA)" <ji...@apache.org>]

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

Mahadev konar commented on ZOOKEEPER-547:
-----------------------------------------

saw this in the logs - 

{code}
junit.framework.AssertionFailedError: fastleaderelection used
	at org.apache.zookeeper.server.quorum.QuorumPeerMainTest.testQuorumDefaults(QuorumPeerMainTest.java:354)
{code}

Doesnt seem to be related to the patch but will have to take a closer look.

> Sanity check in QuorumCnxn Manager and quorum communication port.
> -----------------------------------------------------------------
>
>                 Key: ZOOKEEPER-547
>                 URL: https://issues.apache.org/jira/browse/ZOOKEEPER-547
>             Project: Zookeeper
>          Issue Type: Bug
>          Components: leaderElection, server
>    Affects Versions: 3.2.0, 3.2.1
>            Reporter: Mahadev konar
>            Assignee: Mahadev konar
>             Fix For: 3.2.2, 3.3.0
>
>         Attachments: ZOOKEEPER-547.patch, ZOOKEEPER-547.patch, ZOOKEEPER-547.patch
>
>
> We need to put some sanity checks in QuorumCnxnManager and the other quorum port for rogue clients. Sometimes a clients might get misconfigured and they might send random characters on such ports. We need to make sure that such rogue clients do not bring down the clients and need to put in some sanity checks with respect to packet lengths and deserialization.

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