diff options
| author | Zbigniew Chyla <zbigniew.chyla@nsn.com> | 2015-01-07 11:04:00 +0100 |
|---|---|---|
| committer | Eric S. Raymond <esr@thyrsus.com> | 2015-01-13 07:23:15 -0500 |
| commit | 18d76d6bda59a9eb0a3f062fac91f481babcbdf4 (patch) | |
| tree | bedf2bc28676681e73a8decc42c4c757faa47e2e /driver_ubx.c | |
| parent | 679ad1b39528615dbaeb22bc741b93dc3c3ee4fb (diff) | |
| download | gpsd-18d76d6bda59a9eb0a3f062fac91f481babcbdf4.tar.gz | |
Always use sizeof to get array size
Don't use constant/expression from an array's definition when referring
to its size. Eliminates redundancy and avoids problems when array size
changes. The change doesn't affect generated code.
Diffstat (limited to 'driver_ubx.c')
| -rw-r--r-- | driver_ubx.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/driver_ubx.c b/driver_ubx.c index 25b4ecf2..616dcac7 100644 --- a/driver_ubx.c +++ b/driver_ubx.c @@ -343,7 +343,7 @@ static void ubx_msg_inf(struct gps_device_t *session, unsigned char *buf, size_t if (data_len > MAX_PACKET_LENGTH - 1) data_len = MAX_PACKET_LENGTH - 1; - (void)strlcpy(txtbuf, (char *)buf + UBX_PREFIX_LEN, MAX_PACKET_LENGTH); + (void)strlcpy(txtbuf, (char *)buf + UBX_PREFIX_LEN, sizeof(txtbuf)); txtbuf[data_len] = '\0'; switch (msgid) { case UBX_INF_DEBUG: |