extra: handle the PARAM_DEREFERENCED hook earlier

One thing which happens is that we dereference a parameter and then set it.
The dereference should be processed first.  Otherwise if it's processed
second maybe we set the parameter to NULL, and then processing the
dereference means it can't be NULL so now it's an impossible state.

Also it makes no sense to process dereferences afterwards.  There is
nothing to be gained from that.  The PARAM_SET will hopefully say that it
must be valid and that's the end of that.

Signed-off-by: Dan Carpenter <[email protected]>

Author: Dan Carpenter

smatch_extra.c

@@ -2997,7 +2997,7 @@ void register_smatch_extra_late(int id)
 	add_modification_hook(link_id, &match_link_modify);
 	add_hook(&match_dereferences, DEREF_HOOK);
 	add_hook(&match_pointer_as_array, OP_HOOK);
-	select_return_implies_hook(DEREFERENCE, &set_param_dereferenced);
+	select_return_implies_hook_early(DEREFERENCE, &set_param_dereferenced);
 	add_hook(&match_function_call, FUNCTION_CALL_HOOK);
 	add_hook(&match_assign, ASSIGNMENT_HOOK);
 	add_hook(&match_assign, GLOBAL_ASSIGNMENT_HOOK);