Add more comments on linked attribute limitations

Stefan Metzmacher metze at
Tue Oct 30 09:33:19 UTC 2018

Hi Tim, hi Andrew,

> RB+ with the suggested changes.diff attached. I attached an updated
> version of the original patch too.
> Suggested changes:
> - replmd_delete_internals(): I just moved the comment up a bit. (I found
> it a bit confusing to talk about not finding backlinks when we clearly
> had found one).
> - replmd_process_linked_attribute(): this function is complex enough,
> and replmd_delete_link_value() should already cover the DRS code-path.
> It might be better sticking a comment in linked_attributes.c instead.
> - minor elaboration on "filtering at runtime" and fixed typo.
> On 30/10/18 4:01 PM, Andrew Bartlett wrote:
>> G'Day,
>> While reviewing I
>> forgot how bad the limitations on our linked attribute handling are.

I've included this into my branch.

>> This commit just tries to add some more warnings that might help remind
>> the next person trying to remember why we can't have nice things.
>> Please review so I can push with metze's patches.

Can I assume I can add a review tag to my patches and push them to master?


-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 833 bytes
Desc: OpenPGP digital signature
URL: <>

More information about the samba-technical mailing list