Is that the total fo your code?
As there are no dim statements and no error trapping.
Is this recordset based on a query?
Are the field names spelt correctly?
An example database with a small amount of dummy data would be very helpful.