ConnMan is not quite as strict as the kernel, but maintainers prefer patch descriptions with an imperative tone, so if there is another iteration of the patch I'd change this:
Code
During interface teardown, the GSupplicantInterface resources were not being
properly canceled, leading to a socket file descriptor leak. This patch
ensures that g_supplicant_interface_cancel is executed safely when the
interface is removed.
To this (but with correct line wrapping):
Code
During interface teardown GSupplicantInterface resources are not properly canceled leading to a socket file descriptor leak. Fix this by ensuring g_supplicant_interface_cancel is always executed when an interface is removed.
NB: there appears to be a stray line-delete in the patch:
Diff
--- a/plugins/wifi.c
+++ b/plugins/wifi.c
@@ -2740,19 +2740,17 @@ static void interface_removed(GSupplicantInterface *interface)
DBG("ifname %s", ifname);
wifi = g_supplicant_interface_get_data(interface);
-
if (wifi)
wifi->interface = NULL;
If you have an account on patchwork it would be good to mark earlier iterations of patches as superseded so maintainer tools (and thus maintainers) can ignore them.
To the original question: I'd prefer to see the patch merged and package bumped than place 'known bug' info in the wiki.