diff options
| author | Andre Hartmann <aha_1980@gmx.de> | 2016-08-09 11:47:54 +0200 |
|---|---|---|
| committer | André Hartmann <aha_1980@gmx.de> | 2016-08-26 08:19:35 +0000 |
| commit | 46c3db77f3289cced28e39aefa03f175b61cdcfc (patch) | |
| tree | 11f99393bb1fa493f8244d9f4ff5b44e7b5b021a /src/plugins/canbus/socketcan | |
| parent | 90712efc5b42d31066a388aa370a388c09da6e24 (diff) | |
SocketCAN: Use qstrncpy instead strcpy
* Copies at most sizeof(ifr_name) bytes
* Guarantees that ifr_name is zero-terminated
Change-Id: I68966767002dbe5f349ce9f03f34f4657d1827e9
Reviewed-by: Alex Blasche <alexander.blasche@qt.io>
Diffstat (limited to 'src/plugins/canbus/socketcan')
| -rw-r--r-- | src/plugins/canbus/socketcan/socketcanbackend.cpp | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/plugins/canbus/socketcan/socketcanbackend.cpp b/src/plugins/canbus/socketcan/socketcanbackend.cpp index b902b80..60f36f9 100644 --- a/src/plugins/canbus/socketcan/socketcanbackend.cpp +++ b/src/plugins/canbus/socketcan/socketcanbackend.cpp @@ -265,7 +265,7 @@ bool SocketCanBackend::connectSocket() return false; } - strcpy(interface.ifr_name, canSocketName.toLatin1().data()); + qstrncpy(interface.ifr_name, canSocketName.toLatin1().constData(), sizeof(interface.ifr_name)); if (ioctl(canSocket, SIOCGIFINDEX, &interface) < 0) { setError(qt_error_string(errno), QCanBusDevice::CanBusError::ConnectionError); |