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.