<meta http-equiv="Content-Type" content="text/html; charset=GB18030"><div>Hello,</div><div><br></div><div>In pool_passwd.c:391</div><div>I found the condition ˇ°if (tok)ˇ± is is always true, Because previous condition has already checked that.</div><div><br></div><div>the code is as follows (the code following dose not seem to be used now, But for security, we'd better fix it)</div><div>---------------------------------</div><div><div><div>383&nbsp; &nbsp; &nbsp; &nbsp; &nbsp;/* Get backend user */</div><div>384&nbsp; &nbsp; &nbsp; &nbsp; &nbsp;t = getNextToken(t, &amp;tok);</div><div>385&nbsp; &nbsp; &nbsp; &nbsp; &nbsp;if (tok)</div><div>386&nbsp; &nbsp; &nbsp; &nbsp; &nbsp;{</div><div>387&nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp;/* check if we also have the password */</div><div>388&nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp;char&nbsp; &nbsp; &nbsp; &nbsp;*pwd;</div><div>389</div><div>390&nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp;t = getNextToken(t, &amp;pwd);</div><div>391&nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp;if (tok)</div><div>392&nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp;{</div><div>393&nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp;pwdMapping-&gt;backendUser.password = pwd;</div><div>394&nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp;pwdMapping-&gt;backendUser.userName = tok;</div><div>395&nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp;pwdMapping-&gt;backendUser.passwordType = get_password_type(pwdMapping-&gt;backendUser.password);</div><div>396&nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp;pwdMapping-&gt;mappedUser = true;</div><div>397&nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp;}</div><div>398&nbsp; &nbsp; &nbsp; &nbsp; &nbsp;}</div><div>399&nbsp; &nbsp; &nbsp; &nbsp; &nbsp;break;</div></div><div></div></div><div>---------------------------------</div><div><br></div><div>I think the condition that really need to be checked here is ˇ±if (pwd)ˇ°.</div><div><br></div><div>Best regards</div>