If node revisions is on uc_product will have multiple rows for the same noe. I fixed by amending the query:

$result = db_query("SELECT distinct(node.nid) FROM {uc_products} INNER JOIN {node} ON uc_products.nid = node.nid WHERE node.status = 1");