Attention is currently required from: fixeria, msuraev. neels has posted comments on this change. ( https://gerrit.osmocom.org/c/libosmocore/+/29122 )
Change subject: Revert "Add function to guess AF_UNSPEC address" ...................................................................... Patch Set 1: (1 comment) Patchset: PS1: Max, the difference is between detecting the address kind of a character string, vs detecting the address kind of a sockaddr_storage that has no AF set aka arbitrary binary data. A character string is like "1.2.3.4" or "1:2:3::4" -- obviously you can detect the AF here. If you have a struct sockaddr, you have arbitrary binary data, which is interpreted as IPv4 or IPv6 *completely depending* on the AF kind being set properly. There is otherwise absolutely no way to find the AF kind. You don't even know the actual size of the address data. AF_UNSPEC is unspecified, using it is an error, period. Pau already explained that in CR, too. I am positive that you will be unable to prove to us that this function is useful in any way. If you wrote a test for this function, you should see. Please remove your -1 vote. -- To view, visit https://gerrit.osmocom.org/c/libosmocore/+/29122 To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings Gerrit-Project: libosmocore Gerrit-Branch: master Gerrit-Change-Id: Ifadd614604cf9d0c2ed1a405493c1c3fcb37ae23 Gerrit-Change-Number: 29122 Gerrit-PatchSet: 1 Gerrit-Owner: neels <[email protected]> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: fixeria <[email protected]> Gerrit-Reviewer: laforge <[email protected]> Gerrit-Reviewer: msuraev <[email protected]> Gerrit-CC: pespin <[email protected]> Gerrit-Attention: fixeria <[email protected]> Gerrit-Attention: msuraev <[email protected]> Gerrit-Comment-Date: Wed, 17 Aug 2022 13:53:40 +0000 Gerrit-HasComments: Yes Gerrit-Has-Labels: No Gerrit-MessageType: comment
