From f3ad05347e55de2fd2a500464acde0979df9941f Mon Sep 17 00:00:00 2001 From: Adam Ierymenko Date: Fri, 30 Aug 2013 17:05:43 -0400 Subject: Improve code security posture by replacing sprintf with a safer function. --- node/Demarc.cpp | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) (limited to 'node/Demarc.cpp') 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"); -- cgit v1.2.3