From 93bafb611a58293e0353320a14db298133c9d99a Mon Sep 17 00:00:00 2001 From: Neels Hofmeyr Date: Fri, 13 Jan 2017 03:12:08 +0100 Subject: cosmetic: use osmo_strlcpy() everywhere Shorten some code and make obvious to the reader that the string copy is done in a safe way. Change-Id: I900726cf06d34128db22a3d3d911ee0d1423b1bd --- openbsc/src/libcommon/gsm_subscriber_base.c | 3 ++- openbsc/src/libcommon/gsup_test_client.c | 3 +-- 2 files changed, 3 insertions(+), 3 deletions(-) (limited to 'openbsc/src/libcommon') diff --git a/openbsc/src/libcommon/gsm_subscriber_base.c b/openbsc/src/libcommon/gsm_subscriber_base.c index 1f98cc66c..1ecdee5a5 100644 --- a/openbsc/src/libcommon/gsm_subscriber_base.c +++ b/openbsc/src/libcommon/gsm_subscriber_base.c @@ -28,6 +28,7 @@ #include #include +#include #include #include @@ -115,7 +116,7 @@ struct gsm_subscriber *subscr_get_or_create(struct gsm_subscriber_group *sgrp, if (!subscr) return NULL; - strncpy(subscr->imsi, imsi, sizeof(subscr->imsi)-1); + osmo_strlcpy(subscr->imsi, imsi, sizeof(subscr->imsi)); subscr->group = sgrp; return subscr; } diff --git a/openbsc/src/libcommon/gsup_test_client.c b/openbsc/src/libcommon/gsup_test_client.c index 861212a64..8be4e7aea 100644 --- a/openbsc/src/libcommon/gsup_test_client.c +++ b/openbsc/src/libcommon/gsup_test_client.c @@ -75,8 +75,7 @@ static struct imsi_op *imsi_op_alloc(void *ctx, const char *imsi, return NULL; io = talloc_zero(ctx, struct imsi_op); - strncpy(io->imsi, imsi, sizeof(io->imsi)); - io->imsi[sizeof(io->imsi)-1] = '\0'; + osmo_strlcpy(io->imsi, imsi, sizeof(io->imsi)); io->type = type; io->timer.cb = imsi_op_timer_cb; io->timer.data = io; -- cgit v1.2.3