tree bb7ca03996156f9ec8e45f54c97054f1de244261
parent 0ae8a029cefc5ef843552327e310ae42fbd0b52a
author Karsten Sperling <113487422+ksperling-apple@users.noreply.github.com> 1718239265 +1200
committer GitHub <noreply@github.com> 1718239265 +0000
gpgsig -----BEGIN PGP SIGNATURE-----
 
 wsFcBAABCAAQBQJmakAhCRC1aQ7uu5UhlAAAPkAQAHDgZkjP5PhANpL4+jHRNLMm
 tRliuNzYMJY5BgWALAeA9FbW7f1qgj2m8gaYEy/CQFAiMqENEWb/gK0g54Fa3ES+
 uEB9RUs3CdFPAHvEJIOZlLMRy9e+uAhwt04/ZV7vVXoBEYtE8ZolB34wfaRccir0
 zBO4/KJugvUWujZoxUti6yN3BBbYOlWH/VumElNgV2oaX7OqHuUxoBKNmzkQmD08
 OWTxW0wNea9QOHnJxiW3e34rQORwpyAaUFFFgcv3ukUlAo5TsSpir3dFyMf92VFr
 gK9YKNvY5OeLNXTe5f1oIOcKabzq/pFzHr/MaJQ3WkcV0dzkOQYXUaWqhsg8NV7w
 yxk9fblNdxSS42UUpD2f/rfUJKzU8cajGzMbYdQJwwiloWMOZEexjOhPu/X0IVp/
 /DjcJEe/+J3hGOiB7sf/bDYAZEVVmkD2tvEtx6VzcPsO2Ryu+ybaReT07dqwgmsu
 8P1/a69yIMErLZ8y9FEHC2NoY7khJ2yQ0fIu/m2+JZIV/dijPRXT5UM6A3Ds/iV6
 XPptJ5Aksc5dkf6bQGtOK6NgBWX1Y0AjxDyySSrPydeto6A9aKjZbDXIi/3iVmu6
 EvKwSW7l59Inozlgp7haLng4wRjhqY6wGtCLJv9jwG9FboX5sm9Hf/iACQ/eiJh3
 biBeySWWxsrD7G54+eDR
 =5R1j
 -----END PGP SIGNATURE-----
 

Fix OperationalSessionSetup notifying success with inactive sessions (#33822)

When a success callback marks the session defunct for some reason, other
succcess callbacks should not be called. Implement a GroupedCallbackList to
make this logic possible.

This specific solution is based on the assumption that we don't want to change
the OperationalSessionSetup API which takes success and two variants of failure
callbacks as separate, client-provided Callback objects, with all callbacks
being optional to provide. We also don't want to introduce additional dynamic
allocation within OperationalSessionSetup e.g. to allocate a struct holding the
related callbacks.

The GroupedCallbackList class makes use of the existing prev/next pointers
within the client-allocated Callback objects to capture the grouping
relationship between them.

Co-authored-by: Boris Zbarsky <bzbarsky@apple.com>