Opened 5 years ago

Closed 2 years ago

#15582 closed defect (fixed)

compile warning in test_util.c: In function 'test_util_sscanf'

Reported by: sysfu Owned by: nickm
Priority: Medium Milestone: Tor: 0.3.2.x-final
Component: Core Tor/Tor Version: Tor: 0.2.6.6
Severity: Normal Keywords: compile, test, openbsd 029-backport 030-backport 031-backport
Cc: Actual Points:
Parent ID: Points: medium
Reviewer: dgoulet Sponsor:

Description

Environment:
OpenBSD 5.6 release with Libevent 1.4.14b-stable, OpenSSL LibreSSL 2.1.6 and Zlib 1.2.3


CC src/test/src_test_test-test_util.o

src/test/test_util.c: In function 'test_util_sscanf':
src/test/test_util.c:1843: warning: Array size (20) smaller than format string size (1000000) (arg 3)
src/test/test_util.c:1949: warning: Array size (20) smaller than format string size (1000) (arg 3)

CC src/test/src_test_test-test_helpers.o


Child Tickets

Change History (9)

comment:1 Changed 4 years ago by nickm

Milestone: Tor: 0.2.8.x-final

comment:2 Changed 4 years ago by nickm

Keywords: lorax openbsd added
Milestone: Tor: 0.2.8.x-finalTor: unspecified
Points: medium

comment:3 Changed 2 years ago by nickm

Resolution: worksforme
Severity: Normal
Status: newclosed

This doesn't appear to be happening on openbsd today according to the buildbots. Closing as worksforme, but please reopen if I'm wrong.

comment:4 Changed 2 years ago by cypherpunks

Resolution: worksforme
Status: closedreopened

The builders on https://buildbot.pixelminers.net/ still show these warnings (among other warnings).

Latest build on OpenBSD-amd64: https://buildbot.pixelminers.net/#/builders/5/builds/688
Latest build on OpenBSD-i386: https://buildbot.pixelminers.net/#/builders/9/builds/699

comment:5 Changed 2 years ago by nickm

Milestone: Tor: unspecifiedTor: 0.3.2.x-final

comment:6 Changed 2 years ago by nickm

Keywords: 029-backport 030-backport 031-backport added; lorax removed
Owner: set to nickm
Status: reopenedaccepted

comment:7 Changed 2 years ago by nickm

Status: acceptedneeds_review

easier than I'd thought. See branch bug15582_029.

comment:8 Changed 2 years ago by dgoulet

Reviewer: dgoulet
Status: needs_reviewmerge_ready

lgtm;

comment:9 Changed 2 years ago by nickm

Resolution: fixed
Status: merge_readyclosed

merged to 029 and forward.

Note: See TracTickets for help on using tickets.