Closed Bug 978851 Opened 6 years ago Closed 6 years ago

LoadInfo::UpdateSystemLoad warns about format string in sscanf

Categories

(Core :: WebRTC, defect)

defect
Not set

Tracking

()

RESOLVED FIXED
mozilla30

People

(Reporter: padenot, Assigned: gcp)

Details

Attachments

(1 file, 3 obsolete files)

LoadInfo::UpdateSystemLoad warns about format string error.
This fixes it locally.
Attachment #8384706 - Flags: review?(gpascutto)
https://pastebin.mozilla.org/4457028
Summary: Bug XXXXXX - Fix sscanf warning in LoadMonitor.cpp. r= → LoadInfo::UpdateSystemLoad warns about format string in sscanf
Comment on attachment 8384706 [details] [diff] [review]
Bug XXXXXX - Fix sscanf warning in LoadMonitor.cpp. r=

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

This is broken as the counters are 64-bit. We'd better use this:
http://stackoverflow.com/questions/6993132/format-specifiers-for-uint8-t-uint16-t
Attachment #8384706 - Flags: review?(gpascutto) → review-
This appears to work.
Attachment #8385226 - Flags: review?(gpascutto)
Attachment #8384706 - Attachment is obsolete: true
Attachment #8385226 - Flags: review?(gpascutto) → review+
This should work on Windows.
Attachment #8385264 - Flags: review?(gpascutto)
Attachment #8385226 - Attachment is obsolete: true
Comment on attachment 8385264 [details] [diff] [review]
Fix sscanf warning in LoadMonitor.cpp. r=

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

Reading the header says that the scanf macros aren't actually supported on Windows. Looks like we have to read in those via something else than sscanf.
Attachment #8385264 - Flags: review?(gpascutto) → review-
Assignee: paul → gpascutto
Attachment #8385264 - Attachment is obsolete: true
Attachment #8385385 - Flags: review?(paul)
Comment on attachment 8385385 [details] [diff] [review]
Patch 1. fix-sscanf-warning in LoadMonitor.cpp

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

::: content/media/webrtc/LoadMonitor.cpp
@@ +219,2 @@
>               &user, &nice,
>               &system, &idle) != 4) {

nit: the indentation is now slightly off.
Attachment #8385385 - Flags: review?(paul) → review+
https://hg.mozilla.org/mozilla-central/rev/0f81cbeae0d4
Status: NEW → RESOLVED
Closed: 6 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla30
You need to log in before you can comment on or make changes to this bug.