Methinks not enough. lngEmpID, which seems to be a variable holding the value EmpID from the table, looks like a number. If so, the quotes around Me.cboEmployeeID.Value would be a problem too. You don't need to specify .Value for a combo or text box - it is the default property.
BTW, you can use either fOsUserName (Google it) or the Environ property to get the logging in user's Windows username. Then you don't need all this password stuff and people asking for a reset because they can't remember theirs. You can use these values to record who makes changes or deletions as well if that's useful, whereas a simply password function doesn't do much (and never really did). Hopefully you have at least done some things to keep users out of the tables, otherwise the password function is easy to defeat.
The more we hear silence, the more we begin to think about our value in this universe.
Paraphrase of Professor Brian Cox.