pespin has posted comments on this change. ( https://gerrit.osmocom.org/c/libosmocore/+/14682 )
Change subject: tell ubsan to ignore SUN_LEN ...................................................................... Patch Set 1: (1 comment) https://gerrit.osmocom.org/#/c/14682/1//COMMIT_MSG Commit Message: https://gerrit.osmocom.org/#/c/14682/1//COMMIT_MSG@9 PS1, Line 9: SUN_LEN is awful and ubsan will report this, so let's ignore it. > SUN_LEN(ptr) ((size_t) (((struct sockaddr_un *) 0)->sun_path) + > strlen((ptr)->sun_path)) […] I still don't get why is this reported as a runtime error. Please explain better what the change exactly does (and how), and put that into the commit description. Information about the mentioned runtime error line is also useful. -- To view, visit https://gerrit.osmocom.org/c/libosmocore/+/14682 To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings Gerrit-Project: libosmocore Gerrit-Branch: master Gerrit-Change-Id: I0ff445072f1b46390c9f70b21d61c789e39358d5 Gerrit-Change-Number: 14682 Gerrit-PatchSet: 1 Gerrit-Owner: Hoernchen <[email protected]> Gerrit-Reviewer: Hoernchen <[email protected]> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: laforge <[email protected]> Gerrit-CC: pespin <[email protected]> Gerrit-Comment-Date: Tue, 16 Jul 2019 16:34:53 +0000 Gerrit-HasComments: Yes Gerrit-Has-Labels: No Comment-In-Reply-To: pespin <[email protected]> Comment-In-Reply-To: Hoernchen <[email protected]> Gerrit-MessageType: comment
