You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@thrift.apache.org by ro...@apache.org on 2010/10/18 10:22:58 UTC
svn commit: r1023682 - in /incubator/thrift/trunk/lib/cpp/src/transport:
TServerSocket.cpp TSocket.cpp
Author: roger
Date: Mon Oct 18 08:22:57 2010
New Revision: 1023682
URL: http://svn.apache.org/viewvc?rev=1023682&view=rev
Log:
THRIFT-916 gcc warnings(partial) use snprintf instead of sprintf
Patch: Christian Lavoie
Modified:
incubator/thrift/trunk/lib/cpp/src/transport/TServerSocket.cpp
incubator/thrift/trunk/lib/cpp/src/transport/TSocket.cpp
Modified: incubator/thrift/trunk/lib/cpp/src/transport/TServerSocket.cpp
URL: http://svn.apache.org/viewvc/incubator/thrift/trunk/lib/cpp/src/transport/TServerSocket.cpp?rev=1023682&r1=1023681&r2=1023682&view=diff
==============================================================================
--- incubator/thrift/trunk/lib/cpp/src/transport/TServerSocket.cpp (original)
+++ incubator/thrift/trunk/lib/cpp/src/transport/TServerSocket.cpp Mon Oct 18 08:22:57 2010
@@ -264,7 +264,7 @@ void TServerSocket::listen() {
}
address.sun_family = AF_UNIX;
- sprintf(address.sun_path, path_.c_str());
+ snprintf(address.sun_path, sizeof(address.sun_path), "%s", path_.c_str());
len = sizeof(address);
do {
Modified: incubator/thrift/trunk/lib/cpp/src/transport/TSocket.cpp
URL: http://svn.apache.org/viewvc/incubator/thrift/trunk/lib/cpp/src/transport/TSocket.cpp?rev=1023682&r1=1023681&r2=1023682&view=diff
==============================================================================
--- incubator/thrift/trunk/lib/cpp/src/transport/TSocket.cpp (original)
+++ incubator/thrift/trunk/lib/cpp/src/transport/TSocket.cpp Mon Oct 18 08:22:57 2010
@@ -219,7 +219,7 @@ void TSocket::openConnection(struct addr
}
address.sun_family = AF_UNIX;
- sprintf(address.sun_path, path_.c_str());
+ snprintf(address.sun_path, sizeof(address.sun_path), "%s", path_.c_str());
len = sizeof(address);
ret = connect(socket_, (struct sockaddr *) &address, len);
} else {