Chao Sun created HDFS-14305: ------------------------------- Summary: Serial number in BlockTokenSecretManager could overlap between different namenodes Key: HDFS-14305 URL: https://issues.apache.org/jira/browse/HDFS-14305 Project: Hadoop HDFS Issue Type: Improvement Components: security Reporter: Chao Sun Assignee: Chao Sun
Currently, a {{BlockTokenSecretManager}} starts with a random integer as the initial serial number, and then use this formula to rotate it: {code:java} this.intRange = Integer.MAX_VALUE / numNNs; this.nnRangeStart = intRange * nnIndex; this.serialNo = (this.serialNo % intRange) + (nnRangeStart); {code} while {{numNNs}} is the total number of NameNodes in the cluster, and {{nnIndex}} is the index of the current NameNode specified in the configuration {{dfs.ha.namenodes.<nameservice>}}. However, with this approach, different NameNode could have overlapping ranges for serial number. For simplicity, let's assume {{Integer.MAX_VALUE}} is 100, and we have 2 NameNodes {{nn1}} and {{nn2}} in configuration. Then the ranges for these two are: {code} nn1 -> [-49, 49] nn2 -> [1, 99] {code} This is because the initial serial number could be any negative integer. Moreover, when the keys are updated, the serial number will again be updated with the formula: {code} this.serialNo = (this.serialNo % intRange) + (nnRangeStart); {code} which means the new serial number could be updated to a range that belongs to a different NameNode, thus increasing the chance of collision again. When the collision happens, DataNodes could overwrite an existing key which will cause clients to fail because of {{InvalidToken}} error. -- This message was sent by Atlassian JIRA (v7.6.3#76005) --------------------------------------------------------------------- To unsubscribe, e-mail: hdfs-issues-unsubscr...@hadoop.apache.org For additional commands, e-mail: hdfs-issues-h...@hadoop.apache.org