property != NULL is checked upstream in xfconf-channel.c, and this
probably shouldn't be an assert in xfconf_cache_old_item_new() (which
by itself works with property == NULL)

Name validation in xfconf-backend-perchannel-xml.c however should
probably be turned into normal checks instead of asserts, let's add
these sanity checks anyway.