Most likely, the email address you are reading back is null - so the paramater value is being passed through as a null value and SQL complains.
Two suggestions for you:
1) Don't use the email address as a user id - create a userID column instead and use that to tie your two (or more) tables together. (It can be int and Identity, or GUID - your choice). Emails are not necessarily unique, or unchanging and you want a nice simple value for that field.
2) Never store passwords in clear text - it is a major security risk. There is some information on how to do it here:
Password Storage: How to do it.[
^]
Or alternatively, just implement
Membership[
^] and let the system handle all this - it's a lot easier and works better too!