[FAB-3614] Add chaincode name checking

Add chaincode name checking in chaincodeInstall, with existing path and version checking. Currently, the checking is statically written in hard-code. Futhermore, we may need more checking with other params in future. Change-Id: I4372dd6e05aafecb96bffcf06894d0fc039163cc Signed-off-by: Baohua Yang <[email protected]>
hyperledger · May 3, 2017 · fe4ef55 · fe4ef55
1 parent d6fbfcf
commit fe4ef55
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/peer/chaincode/install.go b/peer/chaincode/install.go
@@ -164,8 +164,8 @@ func chaincodeInstall(cmd *cobra.Command, ccpackfile string, cf *ChaincodeCmdFac
 
 	var ccpackmsg proto.Message
 	if ccpackfile == "" {
-		if chaincodePath == common.UndefinedParamValue || chaincodeVersion == common.UndefinedParamValue {
-			return fmt.Errorf("Must supply value for %s path and version parameters.", chainFuncName)
+		if chaincodePath == common.UndefinedParamValue || chaincodeVersion == common.UndefinedParamValue || chaincodeName == common.UndefinedParamValue {
+			return fmt.Errorf("Must supply value for %s name, path and version parameters.", chainFuncName)
 		}
 		//generate a raw ChaincodeDeploymentSpec
 		ccpackmsg, err = generateChaincode(cmd, chaincodeName, chaincodeVersion)