Merge pull request #1833 from psychocrypt/fix-amdJconfVariableCheck

fix that type of `memChunk` is not tested
fireice-uk · Sep 19, 2018 · 35377a4 · 35377a4
2 parents 55cfa2d + 8a2f294
commit 35377a4
Showing 1 changed file with 3 additions and 3 deletions.
diff --git a/xmrstak/backend/amd/jconf.cpp b/xmrstak/backend/amd/jconf.cpp
@@ -142,14 +142,14 @@ bool jconf::GetThreadConfig(size_t id, thd_cfg &cfg)
 		return false;
 	}
 
-	cfg.memChunk = (int)memChunk->GetInt64();
-
-	if(!idx->IsUint64() || cfg.memChunk > 18 )
+	if(!memChunk->IsUint64() || (int)memChunk->GetInt64() > 18 )
 	{
 		printer::inst()->print_msg(L0, "ERROR: mem_chunk must be smaller than 18");
 		return false;
 	}
 
+	cfg.memChunk = (int)memChunk->GetInt64();
+
 	if(!compMode->IsBool())
 		return false;