> On 24 Oct 2023, at 22:34, Tom Lane <t...@sss.pgh.pa.us> wrote: > > Daniel Gustafsson <dan...@yesql.se> writes: >> I went ahead and applied this on master, thanks for review! Now to see if >> there will be any noticeable difference in resource usage. > > I think that tools like Coverity are likely to whine about your > use of sprintf instead of snprintf. Sure, it's perfectly safe, > but that won't stop the no-sprintf-ever crowd from complaining.
Fair point, that's probably quite likely to happen. I can apply an snprintf() conversion change like this in the two places introduced by this: - sprintf(s, "%d", port); + sprintf(s, sizeof(s), "%d", port); -- Daniel Gustafsson