Attention is currently required from: osmith, msuraev. pespin has posted comments on this change. ( https://gerrit.osmocom.org/c/libosmo-netif/+/30210 )
Change subject: osmux: Introduce API osmux_xfrm_input_set_name() ......................................................................
Patch Set 2:
(1 comment)
File src/osmux_input.c:
https://gerrit.osmocom.org/c/libosmo-netif/+/30210/comment/1157848b_ba86996f PS2, Line 732: link->name = talloc_asprintf(link, "%p", h);
how about using a simple counter instead of using pointers? then we wouldn't need the sed replacemen […]
Max: You are mixing stuff here. One thing is the deployed software which may be difficult to upgrade/modify, and the other is the libosmo-netif implementation in use.
Since this seems to be causing rejection for this patch I can then simply totally drop the pointer and allocate an empty string, or do as osmith suggests, though I don't find it as useful for debuggin purposes as having an address pointer.