Thanks Matt. ________________________________________ From: Matt Borland <[email protected]> Sent: Thursday, November 16, 2023 9:32 AM To: McGrath, Justin M Cc: Boost users list; [email protected] Subject: Re: [Boost-users] Could sprintf be replaced with snprintf?
On Nov 16, 2023, at 4:22 PM, McGrath, Justin M <[email protected]> wrote: Hi Matt, In ODEINT it's called in max_step_checker.hpp. ``` const int m_max_steps; ... char error_msg[200]; std::sprintf(error_msg, "Max number of iterations exceeded (%d).", m_max_steps); ... char error_msg[200]; std::sprintf(error_msg, "Max number of iterations exceeded (%d).", m_max_steps); ``` It looks to me that neither of these uses could possibly overflow, but for whatever reason people have latched onto the idea that sprintf should never be used. Cheers, Justin Justin, It looks like someone filed a PR about a year ago to fix that: https://github.com/boostorg/odeint/pull/58<https://urldefense.com/v3/__https://github.com/boostorg/odeint/pull/58__;!!DZ3fjg!6druIjXAXDKen8dGi26CYtTBh_uBn-gpYisQa9FOdL-EZwBQObAMTh7azQhBLUIgRrZIIjto90zOMbIsnkU$> , but the last commit to ODEINT is spring of 2019. I cc’ed the maintainer so hopefully he sees this. Matt _______________________________________________ Boost-users mailing list [email protected] https://lists.boost.org/mailman/listinfo.cgi/boost-users
