diff options
| author | Adam Ierymenko <adam.ierymenko@gmail.com> | 2013-08-30 17:05:43 -0400 |
|---|---|---|
| committer | Adam Ierymenko <adam.ierymenko@gmail.com> | 2013-08-30 17:05:43 -0400 |
| commit | f3ad05347e55de2fd2a500464acde0979df9941f (patch) | |
| tree | 57376b3114e193201dcfe2d08ef2a5bdb693ad30 /node/Demarc.cpp | |
| parent | 1a7e303f97507842c1a0bfffb27b76277bd43bab (diff) | |
| download | infinitytier-f3ad05347e55de2fd2a500464acde0979df9941f.tar.gz infinitytier-f3ad05347e55de2fd2a500464acde0979df9941f.zip | |
Improve code security posture by replacing sprintf with a safer function.
Diffstat (limited to 'node/Demarc.cpp')
| -rw-r--r-- | node/Demarc.cpp | 5 |
1 files changed, 2 insertions, 3 deletions
diff --git a/node/Demarc.cpp b/node/Demarc.cpp index ddbed6c8..f4060dac 100644 --- a/node/Demarc.cpp +++ b/node/Demarc.cpp @@ -68,15 +68,14 @@ Demarc::~Demarc() } std::string Demarc::describe(Demarc::Port p) - throw() { char buf[64]; switch ((DemarcPortType)(((uint64_t)p) >> 60)) { case PORT_TYPE_UDP_SOCKET_V4: - sprintf(buf,"udp/4/%d",(int)((uint64_t)p & 0xffff)); + Utils::snprintf(buf,sizeof(buf),"udp/4/%d",(int)((uint64_t)p & 0xffff)); return std::string(buf); case PORT_TYPE_UDP_SOCKET_V6: - sprintf(buf,"udp/6/%d",(int)((uint64_t)p & 0xffff)); + Utils::snprintf(buf,sizeof(buf),"udp/6/%d",(int)((uint64_t)p & 0xffff)); return std::string(buf); case PORT_TYPE_LOCAL_ETHERNET: return std::string("ethernet"); |