Validated unique namespace id.

src/keva/main.cpp

@@ -89,8 +89,7 @@ CKevaMemPool::removeConflicts (const CTransaction& tx)
   if (!tx.IsKevacoin ())
     return;
 
-  for (const auto& txout : tx.vout)
+  for (const auto& txout : tx.vout) {
-    {
     const CKevaScript nameOp(txout.scriptPubKey);
     if (nameOp.isKevaOp() && nameOp.getKevaOp() == OP_KEVA_PUT) {
       const valtype& nameSpace = nameOp.getOpNamespace();
@@ -178,6 +177,13 @@ CKevaMemPool::check(const CCoinsView& coins) const
   }
 }
 
+bool CKevaMemPool::validateNamespace(const CTransaction& tx, const valtype& nameSpace) const
+{
+  valtype kevaNamespace = ToByteVector(Hash160(ToByteVector(tx.vin[0].prevout.hash)));
+  kevaNamespace.insert(kevaNamespace.begin(), CKevaScript::NAMESPACE_PREFIX);
+  return kevaNamespace == nameSpace;
+}
+
 bool
 CKevaMemPool::checkTx (const CTransaction& tx) const
 {
@@ -190,20 +196,22 @@ CKevaMemPool::checkTx (const CTransaction& tx) const
      mempool at once (building upon each other).  This is disallowed, though,
      since the current mempool implementation does not like it.  (We keep
      track of only a single update tx for each name.)  */
-
   for (const auto& txout : tx.vout) {
     const CKevaScript nameOp(txout.scriptPubKey);
     if (!nameOp.isKevaOp ())
       continue;
-
     switch (nameOp.getKevaOp ()) {
       case OP_KEVA_NAMESPACE:
       {
         const valtype& nameSpace = nameOp.getOpNamespace();
         std::map<valtype, uint256>::const_iterator mi;
         mi = mapNamespaceRegs.find(nameSpace);
-        if (mi != mapNamespaceRegs.end ())
+        if (mi != mapNamespaceRegs.end ()) {
           return false;
+        }
+        if (!validateNamespace(tx, nameSpace)) {
+          return false;
+        }
         break;
       }
 

src/keva/main.h

@@ -136,6 +136,11 @@ private:
    */
   NamespaceTxMap mapNamespaceUpdates;
 
+  /**
+   * Validate that the namespace is the hash of the first TxIn.
+   */
+  bool validateNamespace(const CTransaction& tx, const valtype& nameSpace) const;
+
 public:
 
   /**

src/script/keva.cpp

@@ -98,7 +98,6 @@ CScript CKevaScript::replaceKevaNamespace(const CScript& oldScript, const uint25
   const valtype& displayName = kevaOp.getOpNamespaceDisplayName();
   kaveNamespace = ToByteVector(Hash160(ToByteVector(txId)));
   kaveNamespace.insert(kaveNamespace.begin(), NAMESPACE_PREFIX);
-  //kaveNamespace = EncodeBase58(kaveNamespaceVal);
   return CKevaScript::buildKevaNamespace(kevaOp.getAddress(), kaveNamespace, displayName);
 }
 

src/wallet/rpckeva.cpp

@@ -72,7 +72,7 @@ UniValue keva_namespace(const JSONRPCRequest& request)
 
   CKeyID keyId = pubKey.GetID();
 
-  // The namespace name is: Hash160("first txin" || displayName)
+  // The namespace name is: Hash160("first txin")
   // For now we don't know the first txin, so use dummy name here.
   // It will be replaced later in CreateTransaction.
   valtype namespaceDummy = ToByteVector(std::string(DUMMY_NAMESPACE));