1295085 - Replace PR_snprintf and s[n]printf calls with SprintfLiteral in netwerk/

Status: RESOLVED FIXED

(Core :: Networking, defect)

Description

[Chris Peterson [:cpeterson]]

Attachment: netwerk-SprintfLiteral.patch

Replace some PR_snprintf and s[n]printf calls with MFBT's SprintfLiteral, a safer sprintf for printing to fixed-size character arrays. SprintfLiteral also requires some int64_t format specifiers to be replaced with standard macros like PRId64 and PRIX64.

Comment 1

[Honza Bambas (:mayhemer)]

May I ask what is the motivation here?  Why is s[n]printf bad?  Why do we again have an overload of ourselves and not use native CRT function?

Comment 2

[Chris Peterson [:cpeterson]]

(In reply to Honza Bambas (:mayhemer) from comment #1)
> May I ask what is the motivation here?  Why is s[n]printf bad?  Why do we
> again have an overload of ourselves and not use native CRT function?

SprintfLiteral will automatically calculate the correct buffer size for character arrays, though the snprintf calls replaced in this patch using sizeof(buffer) were already correct. So changing them from the standard snprintf to a MFBT function might not be worthwhile. I can withdraw those changes, if you prefer to stick with snprintf() and sizeof(buffer).

More importantly, though, this patch also replaces NSPR's PR_snprintf (bug 1197205). Reducing NSPR usage is a worthy goal. :)

Comment 3

[Jason Duell]

Comment on attachment 8781030 [details] [diff] [review]
netwerk-SprintfLiteral.patch

Review of attachment 8781030 [details] [diff] [review]:
-----------------------------------------------------------------

> Reducing NSPR usage is a worthy goal.

+1

Comment 4

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/mozilla-inbound/rev/cf311c1a56f1
Replace PR_snprintf and s[n]printf calls with SprintfLiteral in netwerk/. r=jduell

Comment 5

[Wes Kocher (:KWierso) (Not reading bugmail; email directly if needed)]

https://hg.mozilla.org/mozilla-central/rev/cf311c1a56f1