feedc0de/qt-creator
1
0
Fork 0
forked from qt-creator/qt-creator
Code Pull Requests Activity

tst_QtcProcess: Use snprintf instead of sprintf

Browse Source 
This fixes the following warning:

warning: 'sprintf' is deprecated: This function is provided for
compatibility reasons only.  Due to security concerns inherent in
the design of sprintf(3), it is highly recommended that you use
snprintf(3) instead.

Change-Id: Ic8eac91b02cb60122f603506e083dbdf9eca8712
Reviewed-by: Eike Ziller <eike.ziller@qt.io>
This commit is contained in:
Jarek Kobus
2023-03-06 13:11:15 +01:00
parent ce457a9cdc
commit dc8da57e64
1 changed files with 2 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

4
tests/auto/utils/qtcprocess/tst_qtcprocess.cpp
View File

@@ -698,11 +698,11 @@ void tst_QtcProcess::expandMacros_data()
title = vals[i].in; title = vals[i].in;
} else { } else {
char buf[80]; char buf[80];
sprintf(buf, "%s: %s", title, vals[i].in); snprintf(buf, 80, "%s: %s", title, vals[i].in);
QTest::newRow(buf) << QString::fromLatin1(vals[i].in) QTest::newRow(buf) << QString::fromLatin1(vals[i].in)
<< QString::fromLatin1(vals[i].out) << QString::fromLatin1(vals[i].out)
<< vals[i].os; << vals[i].os;
sprintf(buf, "padded %s: %s", title, vals[i].in); snprintf(buf, 80, "padded %s: %s", title, vals[i].in);
QTest::newRow(buf) << QString(sp + QString::fromLatin1(vals[i].in) + sp) QTest::newRow(buf) << QString(sp + QString::fromLatin1(vals[i].in) + sp)
<< QString(sp + QString::fromLatin1(vals[i].out) + sp) << QString(sp + QString::fromLatin1(vals[i].out) + sp)
<< vals[i].os; << vals[i].os;