common: and_null_if_valid + on_error improvements

[FSMaxB]

It is to hard to split these commits, so both in one:

• replace all _and_null macros with _and_null_if_valid that check if a pointer is NULL before freeing anything
  • This has probably fixed a lot of bugs that could occur when an allocation failure happens and it has reduced the code size and visual complexity
• replace if (status.status == SUCCESS) with on_error
• use only ")" and not ");" for multiline macros