astitcher commented on issue #176: NO-JIRA: [c] Fix Coverity warning of buffer overrun in pn_proactor_addr URL: https://github.com/apache/qpid-proton/pull/176#issuecomment-465636973 I don't think this is the correct fix. There is no problem using strncat that requires inventing a new (and possibly wrong) alternative that does the same thing but perhaps a bit different. In any case as far as I can see there isn't a buffer overrun problem with the fixed code at all! pn_proactor_addr can only overrun the buffer it's given if you give it the wrong length.
---------------------------------------------------------------- This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org With regards, Apache Git Services --------------------------------------------------------------------- To unsubscribe, e-mail: dev-unsubscr...@qpid.apache.org For additional commands, e-mail: dev-h...@qpid.apache.org
