389-ds-base

#50563 Issue 49324 - idl_new report index name in error conditions
Closed by spichugi. Opened by mreynolds.
mreynolds/389-ds-base issue49324  into  master

Download 50563.patch

Description: Add the index attribute name to error messages

relates: https://pagure.io/389-ds-base/issue/49324

Is there ever a case where key.data or the attribute would be NULL?

I think I'd like some PR_ASSERT() on the value that's passed to the %s format, but that's just paranoia. Otherwise I think this is a good change.

rebased onto f47b2d8b8abedff135ad34c189b0b5bfa5fd0140

Is there ever a case where key.data or the attribute would be NULL?
I think I'd like some PR_ASSERT() on the value that's passed to the %s format, but that's just paranoia. Otherwise I think this is a good change.

@firstyear - Added a PR_ASSERT to the top of the function, please review...

LGTM as well.

rebased onto 441d5abaf56385bf010b66df4caab099b7a6349c

Pull-Request has been merged by mreynolds

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

This pull request has been cloned to Github as issue and is available here:
- https://github.com/389ds/389-ds-base/issues/3619

If you want to continue to work on the PR, please navigate to the github issue,
download the patch from the attachments and file a new pull request.

Thank you for understanding. We apologize for all inconvenience.

Pull-Request has been closed by spichugi
Metadata
None
No Tags
Static archive. pagure.io was sunset in 2026; this is a read-only snapshot.
Red Hat Logo

Fedora is sponsored by Red Hat. Learn more about the relationship between Red Hat and Fedora »

© Red Hat, Inc. and others.