Hi,
The attached two patches are related to:
https://fedorahosted.org/sssd/ticket/2257
The first patch is included pretty much for completeness, as I noted
during development of the unit test, the blob type didn't handle any
default value.
The second patch directly addresses #2257. The previous code didn't
handle copying options well if the option was set to zero, because the
code followed logic like:
if (oldval) {
newval = oldval;
else {
newval = defval;
}
The patch implements Sumit's idea to provide a separate function for
copying default values and amend the generic copy function to only
create a copy using the current values.