Closed Bug 1027004 Opened 10 years ago Closed 10 years ago

Add newline to nsSocketTransport2.cpp fprintf_stderr()

Categories

(Core :: Networking: DNS, defect)

defect
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla33

People

(Reporter: emorley, Assigned: emorley)

References

Details

Attachments

(1 file)

In some of the log output the newline is missing, others not. I'm guessing by luck/platform we're adding a newline - but imagine we should be doing so ourselves when using fprintf_stderr()

eg:
https://tbpl.mozilla.org/php/getParsedLog.php?id=41949590&tree=Try#error0
03:31:48     INFO -  Automation Error: Non-local network connections are disabled and a connection attempt to safebrowsing.google.com (74.125.239.98) was made.  You should only access hostnames available via the test networking proxy (if running mochitests) or from a test-specific httpd.js server (if running xpcshell tests).  Browser services should be disabled or redirected to a local server.1403087268647	Marionette	INFO	loaded marionette-listener.js
Comment on attachment 8442023 [details] [diff] [review]
Add newline to MOZ_DISABLE_NONLOCAL_CONNECTIONS fprintf_stderr()

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

Doh!
Attachment #8442023 - Flags: review?(nfroyd) → review+
https://hg.mozilla.org/mozilla-central/rev/6894b106f39e
Status: ASSIGNED → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla33
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: