diff options
| author | Harald Welte <laforge@gnumonks.org> | 2018-08-11 14:15:09 +0200 |
|---|---|---|
| committer | Harald Welte <laforge@gnumonks.org> | 2018-08-11 16:10:31 +0200 |
| commit | d4fb4fdea097f0fd1e9f2cd72d0d8a30c91f48bf (patch) | |
| tree | 2a76d28f1dfc3ad338cec98d08c66fd1cf826b61 /src/host/layer23/src/mobile/gsm411_sms.c | |
| parent | 1d68468636e789555bee99e1d040b0648d1f02c3 (diff) | |
layer23: Replace all instances of strncpy() by osmo_strlcpy
This gives us working/safe zero termination without overflowing
the destination string size.
Change-Id: Ica6098ceba2bd01ce3b216085442cc5eed0ca507
Diffstat (limited to 'src/host/layer23/src/mobile/gsm411_sms.c')
| -rw-r--r-- | src/host/layer23/src/mobile/gsm411_sms.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/src/host/layer23/src/mobile/gsm411_sms.c b/src/host/layer23/src/mobile/gsm411_sms.c index 623ba538..c476ff40 100644 --- a/src/host/layer23/src/mobile/gsm411_sms.c +++ b/src/host/layer23/src/mobile/gsm411_sms.c @@ -104,7 +104,7 @@ struct gsm_sms *sms_from_text(const char *receiver, int dcs, const char *text) if (!sms) return NULL; - strncpy(sms->text, text, sizeof(sms->text)-1); + OSMO_STRLCPY_ARRAY(sms->text, text); /* FIXME: don't use ID 1 static */ sms->reply_path_req = 0; @@ -112,7 +112,7 @@ struct gsm_sms *sms_from_text(const char *receiver, int dcs, const char *text) sms->ud_hdr_ind = 0; sms->protocol_id = 0; /* implicit */ sms->data_coding_scheme = dcs; - strncpy(sms->address, receiver, sizeof(sms->address)-1); + OSMO_STRLCPY_ARRAY(sms->address, receiver); /* Generate user_data */ sms->user_data_len = gsm_7bit_encode_n(sms->user_data, sizeof(sms->user_data), sms->text, NULL); |