Have you printed the statement out, say using Debug.Print, with the values from the combobox before it is executed? What does it look like then? It looks a little strange here. I'd like to see the values first.