Don't assert(pattern), it is legit to be NULL

If ill-formed regex is passed (e.g. from policy), it is wrong
to assert()
Don't use Attribute-laden ItemList save/load functions

/etc/fstab and .log files are not under edit_file editing
model, so their load/save should not be influenced by Attributes
struct.
Move RawSaveItemList to the module it is declared in
Fix return type and argument name for accurateness
Remove check that argument to -f should be at least 5 characters long

This check is wrong (a.cf is a valid name for a policy file) and
is not applied consistently (cf-monitord and cf-runagent were exempt).
if (foo) {} is not a very useful construct