[tor-bugs] #24861 [Core Tor/Tor]: Using %zu seems to break mingw :/

Tor Bug Tracker & Wiki blackhole at torproject.org
Wed Jan 10 18:46:04 UTC 2018


#24861: Using %zu seems to break mingw :/
------------------------------+--------------------------------------------
     Reporter:  nickm         |      Owner:  (none)
         Type:  defect        |     Status:  new
     Priority:  High          |  Milestone:  Tor: 0.3.3.x-final
    Component:  Core Tor/Tor  |    Version:
     Severity:  Normal        |   Keywords:  mingw compatibility regression
Actual Points:                |  Parent ID:
       Points:                |   Reviewer:
      Sponsor:                |
------------------------------+--------------------------------------------
 It looks like we found out why we weren't using `%zu` before:
 {{{
 17:55:39 In file included from src/or/or.h:72:0,
 17:55:39                  from src/or/circuitlist.c:54:
 17:55:39 src/or/circuitlist.c: In function 'circuits_handle_oom':
 17:55:39 src/or/circuitlist.c:2407:26: error: unknown conversion type
 character 'z' in format [-Werror=format=]
 17:55:39    log_notice(LD_GENERAL, "We're low on memory (cell queues total
 alloc: %zu,"
 17:55:39                           ^
 17:55:39 ./src/common/torlog.h:232:45: note: in definition of macro
 'log_notice'
 17:55:39    log_fn_(LOG_NOTICE, domain, __FUNCTION__, args, ##__VA_ARGS__)
 17:55:39                                              ^~~~
 17:55:39 src/or/circuitlist.c:2407:26: error: unknown conversion type
 character 'z' in format [-Werror=format=]
 17:55:39    log_notice(LD_GENERAL, "We're low on memory (cell queues total
 alloc: %zu,"
 17:55:39                           ^
 }}}
 (from https://jenkins.torproject.org/job/tor-ci-mingwcross-
 master/1577/ARCHITECTURE=amd64,SUITE=stretch/consoleFull)

 In theory we can select a more c99 one with `__USE_MINGW_ANSI_STDIO`, but
 that would change our stdio everywhere. (Which is a little scary.)

 We could also define a PRIsz macro that has the correct format for
 whatever compiler we are using.  That might be a better choice.

--
Ticket URL: <https://trac.torproject.org/projects/tor/ticket/24861>
Tor Bug Tracker & Wiki <https://trac.torproject.org/>
The Tor Project: anonymity online


More information about the tor-bugs mailing list