de uma olhada em https://wiki.multitheftauto.com/wiki/OnPlayerVehicleEnter
veja que o primeiro argumento ali onde vc deixou 'player' é na verdade o veiculo que foi entrado, pra acessar o player use source pra esse evento, isso tbm ta especificado na wiki, outro detalhe que sugiro adicionar é fazer a checagem só pro banco do motorista, senão quem entrar de passageiro sem cnh vai desligar o carro..
function desligarCarro(carro,assento)
if not getElementData(source, "cnh") and assento==0 then--se nao tiver elementdata cnh e estiver no banco do motorista (assento 0)
setVehicleEngineState(carro, false)
end
end
addEventHandler("onPlayerVehicleEnter" , getRootElement (), desligarCarro)