Incorrect code in affinity setting code ?

On line 90 of engine.c we declare the global variable entry_affinity but in engine_pin() and engine_init() we declare variables with the same name, effectively shadowing the global one. Am I right in thinking that this is incorrect ? Or if it is, shouldn't we rename the locals to avoid confusion ?