Program: 49050
Project: 49050
Buggy source:
/** 
 * Validate nodeGroupCreates member formats and values in the ClusterCreate.
 */
public void validateClusterCreate(List<String> failedMsgList,List<String> warningMsgList,final List<String> distroRoles){
  boolean namenodeHACheck=false;
  int jobtrackerCount=0, hbasemasterCount=0, zookeeperCount=0, workerCount=0, numOfJournalNode=0;
  NodeGroupCreate[] nodeGroupCreates=getNodeGroups();
  if (nodeGroupCreates == null || nodeGroupCreates.length == 0) {
    failedMsgList.add(Constants.MULTI_INPUTS_CHECK);
    return;
  }
 else {
    if (hasHDFSUrlConfigured() && !validateHDFSUrl()) {
      failedMsgList.add(new StringBuilder().append("String_Node_Str").append(getExternalHDFS()).toString());
    }
    validateNodeGroupPlacementPolicies(failedMsgList,warningMsgList);
    validateNodeGroupRoles(failedMsgList);
    validateTempfs(failedMsgList);
    for (    NodeGroupCreate nodeGroupCreate : nodeGroupCreates) {
      checkInstanceNum(nodeGroupCreate,failedMsgList);
      checkNodeGroupRoles(nodeGroupCreate,distroRoles,failedMsgList);
      List<NodeGroupRole> groupRoles=getNodeGroupRoles(nodeGroupCreate);
      if (groupRoles != null) {
        for (        NodeGroupRole role : groupRoles) {
switch (role) {
case MASTER:
            int numOfInstance=nodeGroupCreate.getInstanceNum();
          if (numOfInstance >= 0 && numOfInstance != 1) {
            if (numOfInstance != 2) {
              collectInstanceNumInvalidateMsg(nodeGroupCreate,failedMsgList);
            }
 else {
              namenodeHACheck=true;
            }
          }
        break;
case JOB_TRACKER:
      jobtrackerCount++;
    if (nodeGroupCreate.getInstanceNum() >= 0 && nodeGroupCreate.getInstanceNum() != 1) {
      failedMsgList.add(Constants.WRONG_NUM_OF_JOBTRACKER);
    }
  break;
case HBASE_MASTER:
hbasemasterCount++;
if (nodeGroupCreate.getInstanceNum() == 0) {
collectInstanceNumInvalidateMsg(nodeGroupCreate,failedMsgList);
}
break;
case ZOOKEEPER:
zookeeperCount++;
if (nodeGroupCreate.getInstanceNum() > 0 && nodeGroupCreate.getInstanceNum() < 3) {
failedMsgList.add(Constants.WRONG_NUM_OF_ZOOKEEPER);
}
 else if (nodeGroupCreate.getInstanceNum() > 0 && nodeGroupCreate.getInstanceNum() % 2 == 0) {
warningMsgList.add(Constants.ODD_NUM_OF_ZOOKEEPER);
}
break;
case JOURNAL_NODE:
numOfJournalNode+=nodeGroupCreate.getInstanceNum();
if (nodeGroupCreate.getRoles().contains(HadoopRole.HADOOP_DATANODE.toString()) || nodeGroupCreate.getRoles().contains(HadoopRole.HADOOP_CLIENT_ROLE.toString())) {
failedMsgList.add(Constants.DATA_CLIENT_NODE_JOURNALNODE_COEXIST);
}
break;
case WORKER:
workerCount++;
if (nodeGroupCreate.getInstanceNum() == 0) {
collectInstanceNumInvalidateMsg(nodeGroupCreate,failedMsgList);
}
 else if (isHAFlag(nodeGroupCreate)) {
warningMsgList.add(Constants.WORKER_CLIENT_HA_FLAG);
}
List<String> roles=nodeGroupCreate.getRoles();
if (roles.contains(HadoopRole.HBASE_REGIONSERVER_ROLE.toString()) && !roles.contains(HadoopRole.HADOOP_DATANODE.toString())) {
warningMsgList.add(Constants.REGISONSERVER_DATANODE_SEPERATION);
}
break;
case CLIENT:
if (isHAFlag(nodeGroupCreate)) {
warningMsgList.add(Constants.WORKER_CLIENT_HA_FLAG);
}
break;
case NONE:
warningMsgList.add(Constants.NOT_DEFINED_ROLE);
break;
default :
}
}
}
}
if (namenodeHACheck) {
if (numOfJournalNode >= 0 && numOfJournalNode < 3) {
failedMsgList.add(Constants.WRONG_NUM_OF_JOURNALNODE);
}
 else if (numOfJournalNode > 0 && numOfJournalNode % 2 == 0) {
warningMsgList.add(Constants.ODD_NUM_OF_JOURNALNODE);
}
if (zookeeperCount == 0) {
failedMsgList.add(Constants.NAMENODE_AUTO_FAILOVER_ZOOKEEPER);
}
}
if ((jobtrackerCount > 1) || (zookeeperCount > 1) || (hbasemasterCount > 1)) {
failedMsgList.add(Constants.WRONG_NUM_OF_NODEGROUPS);
}
if (numOfJournalNode > 0 && !namenodeHACheck) {
failedMsgList.add(Constants.NO_NAMENODE_HA);
}
if (workerCount == 0) {
warningMsgList.add(Constants.WRONG_NUM_OF_WORKERNODES);
}
}
}

Fixed version (ground truth):
/** 
 * Validate nodeGroupCreates member formats and values in the ClusterCreate.
 */
public void validateClusterCreate(List<String> failedMsgList,List<String> warningMsgList,final List<String> distroRoles){
  boolean namenodeHACheck=false;
  int masterCount=0, jobtrackerCount=0, hbasemasterCount=0, zookeeperCount=0, workerCount=0, numOfJournalNode=0;
  NodeGroupCreate[] nodeGroupCreates=getNodeGroups();
  if (nodeGroupCreates == null || nodeGroupCreates.length == 0) {
    failedMsgList.add(Constants.MULTI_INPUTS_CHECK);
    return;
  }
 else {
    if (hasHDFSUrlConfigured() && !validateHDFSUrl()) {
      failedMsgList.add(new StringBuilder().append("String_Node_Str").append(getExternalHDFS()).toString());
    }
    validateNodeGroupPlacementPolicies(failedMsgList,warningMsgList);
    validateNodeGroupRoles(failedMsgList);
    validateTempfs(failedMsgList);
    for (    NodeGroupCreate nodeGroupCreate : nodeGroupCreates) {
      checkInstanceNum(nodeGroupCreate,failedMsgList);
      checkNodeGroupRoles(nodeGroupCreate,distroRoles,failedMsgList);
      List<NodeGroupRole> groupRoles=getNodeGroupRoles(nodeGroupCreate);
      if (groupRoles != null) {
        for (        NodeGroupRole role : groupRoles) {
switch (role) {
case MASTER:
            masterCount++;
          int numOfInstance=nodeGroupCreate.getInstanceNum();
        if (numOfInstance >= 0 && numOfInstance != 1) {
          if (numOfInstance != 2) {
            collectInstanceNumInvalidateMsg(nodeGroupCreate,failedMsgList);
          }
 else {
            namenodeHACheck=true;
          }
        }
      break;
case JOB_TRACKER:
    jobtrackerCount++;
  if (nodeGroupCreate.getInstanceNum() >= 0 && nodeGroupCreate.getInstanceNum() != 1) {
    failedMsgList.add(Constants.WRONG_NUM_OF_JOBTRACKER);
  }
break;
case HBASE_MASTER:
hbasemasterCount++;
if (nodeGroupCreate.getInstanceNum() == 0) {
collectInstanceNumInvalidateMsg(nodeGroupCreate,failedMsgList);
}
break;
case ZOOKEEPER:
zookeeperCount++;
if (nodeGroupCreate.getInstanceNum() > 0 && nodeGroupCreate.getInstanceNum() < 3) {
failedMsgList.add(Constants.WRONG_NUM_OF_ZOOKEEPER);
}
 else if (nodeGroupCreate.getInstanceNum() > 0 && nodeGroupCreate.getInstanceNum() % 2 == 0) {
warningMsgList.add(Constants.ODD_NUM_OF_ZOOKEEPER);
}
break;
case JOURNAL_NODE:
numOfJournalNode+=nodeGroupCreate.getInstanceNum();
if (nodeGroupCreate.getRoles().contains(HadoopRole.HADOOP_DATANODE.toString()) || nodeGroupCreate.getRoles().contains(HadoopRole.HADOOP_CLIENT_ROLE.toString())) {
failedMsgList.add(Constants.DATA_CLIENT_NODE_JOURNALNODE_COEXIST);
}
break;
case WORKER:
workerCount++;
if (nodeGroupCreate.getInstanceNum() == 0) {
collectInstanceNumInvalidateMsg(nodeGroupCreate,failedMsgList);
}
 else if (isHAFlag(nodeGroupCreate)) {
warningMsgList.add(Constants.WORKER_CLIENT_HA_FLAG);
}
List<String> roles=nodeGroupCreate.getRoles();
if (roles.contains(HadoopRole.HBASE_REGIONSERVER_ROLE.toString()) && !roles.contains(HadoopRole.HADOOP_DATANODE.toString())) {
warningMsgList.add(Constants.REGISONSERVER_DATANODE_SEPERATION);
}
break;
case CLIENT:
if (isHAFlag(nodeGroupCreate)) {
warningMsgList.add(Constants.WORKER_CLIENT_HA_FLAG);
}
break;
case NONE:
warningMsgList.add(Constants.NOT_DEFINED_ROLE);
break;
default :
}
}
}
}
if (!supportedWithHdfs2()) {
if (namenodeHACheck || masterCount > 1) {
failedMsgList.add(Constants.CURRENT_DISTRO_CAN_NOT_SUPPORT_HDFS2);
}
}
 else if (namenodeHACheck) {
if (numOfJournalNode >= 0 && numOfJournalNode < 3) {
failedMsgList.add(Constants.WRONG_NUM_OF_JOURNALNODE);
}
 else if (numOfJournalNode > 0 && numOfJournalNode % 2 == 0) {
warningMsgList.add(Constants.ODD_NUM_OF_JOURNALNODE);
}
if (zookeeperCount == 0) {
failedMsgList.add(Constants.NAMENODE_AUTO_FAILOVER_ZOOKEEPER);
}
}
if ((jobtrackerCount > 1) || (zookeeperCount > 1) || (hbasemasterCount > 1)) {
failedMsgList.add(Constants.WRONG_NUM_OF_NODEGROUPS);
}
if (workerCount == 0) {
warningMsgList.add(Constants.WRONG_NUM_OF_WORKERNODES);
}
if (numOfJournalNode > 0 && !namenodeHACheck) {
failedMsgList.add(Constants.NO_NAMENODE_HA);
}
}
}