#3084 DP: Don't pass empty string, but NULL to providers
Closed: Fixed Opened by jhrozek.

Since the DP refactoring, we don't send NULL to providers, but an empty string. This breaks some corner cases in providers, like lookups of services by protocol.

Pavel suggested to amend check_and_parse_filter() in dp_target_id.c so it assign NULL instead of "" in data->filter_value (and other) using SBUS_SET_STRING().

We should do this in 1.14.1


When working on this ticket, please also revert changes in aa58e216c1f794bd335151f19e79adbb3ddf4c73

Fields changed

milestone: NEEDS_TRIAGE => SSSD 1.14.1

Fields changed

owner: somebody => pcech

Fields changed

status: new => assigned

Fields changed

patch: 0 => 1

Fields changed

rhbz: => 0

master:

  • dc30c60f166ad9adc63a47a1013508a71624ac87
  • e4ba22c165be4b0ccd0e1b9fc92f2f1e35f78a82

resolution: => fixed
status: assigned => closed

Additional patch
f9d363f9feb7d057d66ba98337e0f8a6fdbca5ab

Metadata Update from @jhrozek:
- Issue assigned to pcech
- Issue set to the milestone: SSSD 1.14.1

SSSD is moving from Pagure to Github. This means that new issues and pull requests
will be accepted only in SSSD's github repository.

This issue has been cloned to Github and is available here:
- https://github.com/SSSD/sssd/issues/4117

If you want to receive further updates on the issue, please navigate to the github issue
and click on subscribe button.

Thank you for understanding. We apologize for all inconvenience.

Metadata