Attention is currently required from: pespin, dexter.
msuraev has posted comments on this change. (
https://gerrit.osmocom.org/c/libosmocore/+/28995 )
Change subject: Make osmo_sockaddr_str_from_sockaddr() less picky
......................................................................
Patch Set 7:
(1 comment)
File src/sockaddr_str.c:
https://gerrit.osmocom.org/c/libosmocore/+/28995/comment/befb4ad9_399120e4
PS7, Line 389: guess = (src->ss_family == AF_UNSPEC) ?
osmo_sockaddr_guess_unspec((const struct sockaddr *)src) : src->ss_family;
This all looks wrong to me, or I'm missing
soemthing. […]
Not "random stuff" but the content withdots or colons which
passed through osmo_sockaddr_guess_unspec(). The random stuff would still likely get
AF_UNSPEC and won't be converted/printed. As the comment says: we relax restrictions,
we do not remove them completely.
--
To view, visit
https://gerrit.osmocom.org/c/libosmocore/+/28995
To unsubscribe, or for help writing mail filters, visit
https://gerrit.osmocom.org/settings
Gerrit-Project: libosmocore
Gerrit-Branch: master
Gerrit-Change-Id: I36f20701663c3c7eae7fedc6551da44800b325bf
Gerrit-Change-Number: 28995
Gerrit-PatchSet: 7
Gerrit-Owner: msuraev <msuraev(a)sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: laforge <laforge(a)osmocom.org>
Gerrit-Reviewer: pespin <pespin(a)sysmocom.de>
Gerrit-CC: dexter <pmaier(a)sysmocom.de>
Gerrit-Attention: pespin <pespin(a)sysmocom.de>
Gerrit-Attention: dexter <pmaier(a)sysmocom.de>
Gerrit-Comment-Date: Sat, 13 Aug 2022 09:14:41 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Comment-In-Reply-To: pespin <pespin(a)sysmocom.de>
Gerrit-MessageType: comment