大伙帮我看下以下代码错在哪？关于验证密码和验证码的

我想验证密码不能为空，关键代码如下：
function ckPw1(password){
       alert(password);
       if(password != ""){
          document.getElementById("dvPw1").className = 'correct';
          return true;
       }else{
          document.getElementById("dvPw1").className = 'err';
          return false;
       }
    }
注：其中correct和err为css样式
<tr>
  <td width=30% height=50px align=right>密&nbsp;&nbsp;码</td>
  <td width=40% align=left><input type=password name=password id=password autocomplete=off onkeyup=ckPw1(this.value)/></td>
  <td><div id=dvPw1>密码不能为空</div></td>
</tr>

另外同样的道理，我想验证验证码填写是否正确，关键代码如下：
function ckPw1(validcode){
       alert(validcode);
    var validcode2=document.all.image.value;
       if(validcode==validcode2){
          document.getElementById("validcode").className = 'correct';
          return true;
       }else{
          document.getElementById("validcode").className = 'err';
          return false;
       }
    }

<tr>
  <td width=30% height=50px align=right>校验码</td>
  <td width=40% align=left><input type=text name=validCode autocomplete=off id=validCode  autocomplete=off onkeyup=ckPw1(this.value)/></td>
  <td><img id=image name=image src="ajax/getValidateImage" /><a href="javascript:changeImage()">换一张</a></td>
  <td><div id=validcode>检验检验码</div></td>
</tr>
是在看不出错在哪里了，求教各位大虾！
----------------解决方案--------------------------------------------------------
你那验证码明显是一张图片，而且看样子应该是用servlet生成的。。。如果是这样的话在生成验证码的时候那些应该把这验证码都保存在了session或cookie中。如果要判断是否相等应该是判断输入的值跟session或cookie中的验证码是否相等。用javascript或取图片的value是不行的。

----------------解决方案--------------------------------------------------------