[tbb-bugs] #23663 [Applications/Tor Browser]: ESR52 codebase is incompatible with anything below Universal C Runtime (CRT) in Windows
Tor Bug Tracker & Wiki
blackhole at torproject.org
Tue Sep 26 20:08:58 UTC 2017
#23663: ESR52 codebase is incompatible with anything below Universal C Runtime
(CRT) in Windows
--------------------------------------+-----------------------------------
Reporter: cypherpunks | Owner: tbb-team
Type: defect | Status: needs_information
Priority: High | Milestone:
Component: Applications/Tor Browser | Version:
Severity: Major | Resolution:
Keywords: tbb-security | Actual Points:
Parent ID: | Points:
Reviewer: | Sponsor:
--------------------------------------+-----------------------------------
Changes (by gk):
* status: new => needs_information
Comment:
Replying to [comment:4 cypherpunks]:
> Don't you see that Jacek's patch activated compat shims for mingw? They
were removed later as useless for UCRT (but needed for <=
`msvcr120.dll`!).
Oh, okay. You are just concerned about https://hg.mozilla.org/mozilla-
central/rev/5680a55b2ec1? I thought about cases in the other patches as
well as you posted them in the description. But as I said they are guarded
by `_MSC_VER` defines which are not used by mingw-w64 anyway.
So it seems
{{{
-if CONFIG['OS_ARCH'] == 'WINNT':
- SOURCES += [
- '../compat/strtod.c'
}}}
is the thing that is bothering you. Back then this got introduced to fix
compilation with mingw-w64. But that's not an issue anymore without this
particular code. So, what exactly is the problem with that removal for our
mingw-w64 builds as they are building fine now? And could you point to the
security problematic that you think is obvious with removing those three
code lines? (the one you mentioned in comment:2 does not seem to be it)
--
Ticket URL: <https://trac.torproject.org/projects/tor/ticket/23663#comment:5>
Tor Bug Tracker & Wiki <https://trac.torproject.org/>
The Tor Project: anonymity online
More information about the tbb-bugs
mailing list