I don't think this function should return anything.  Is the return value used somewhere?