neels submitted this change.

View Change

Approvals: Jenkins Builder: Verified neels: Looks good to me, approved
follow-up to osmo_sockaddr_from/to_octets()

Forgot to apply one code review change before merge.

Related: OS#5599
Change-Id: Id202f3fda51bea9149ab3462a2efe35cf08d8030
---
M src/socket.c
1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/src/socket.c b/src/socket.c
index 6e5bb46..80cba17 100644
--- a/src/socket.c
+++ b/src/socket.c
@@ -1744,7 +1744,7 @@
{
const void *addr;
size_t len;
- switch (os->u.sin.sin_family) {
+ switch (os->u.sa.sa_family) {
case AF_INET:
addr = &os->u.sin.sin_addr;
len = sizeof(os->u.sin.sin_addr);
@@ -1776,7 +1776,7 @@
*os = (struct osmo_sockaddr){0};
switch (src_len) {
case sizeof(os->u.sin.sin_addr):
- os->u.sin.sin_family = AF_INET;
+ os->u.sa.sa_family = AF_INET;
addr = &os->u.sin.sin_addr;
len = sizeof(os->u.sin.sin_addr);
break;

To view, visit change 27085. To unsubscribe, or for help writing mail filters, visit settings.

Gerrit-Project: libosmocore
Gerrit-Branch: master
Gerrit-Change-Id: Id202f3fda51bea9149ab3462a2efe35cf08d8030
Gerrit-Change-Number: 27085
Gerrit-PatchSet: 1
Gerrit-Owner: neels <nhofmeyr@sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: neels <nhofmeyr@sysmocom.de>
Gerrit-MessageType: merged