Opened 8 months ago

Closed 7 months ago

#29059 closed defect (implemented)

shellcheck: fuzz_static_testcases.sh issues

Reported by: rl1987 Owned by: rl1987
Priority: Medium Milestone: Tor: 0.4.1.x-final
Component: Core Tor/Tor Version:
Severity: Normal Keywords: technical-debt, nickm-merge
Cc: Actual Points:
Parent ID: Points:
Reviewer: teor Sponsor:

Description

In fuzz_static_testcases.sh line 17:
    f=`basename $fuzzer`
      ^-- SC2006: Use $(..) instead of legacy `..`.
                ^-- SC2086: Double quote to prevent globbing and word splitting.

Child Tickets

Change History (8)

comment:1 Changed 8 months ago by rl1987

Owner: set to rl1987
Status: newaccepted

comment:2 Changed 8 months ago by rl1987

Summary: shellcheck: fuzz_static_testcases.sh issueshellcheck: fuzz_static_testcases.sh issues

comment:3 Changed 8 months ago by rl1987

Status: acceptedneeds_review

comment:4 Changed 8 months ago by nickm

Milestone: Tor: 0.4.1.x-final

comment:5 Changed 7 months ago by dgoulet

Reviewer: teor

comment:6 Changed 7 months ago by teor

Status: needs_reviewmerge_ready

Looks fine to me.

comment:7 Changed 7 months ago by nickm

Keywords: nickm-merge added

comment:8 Changed 7 months ago by nickm

Resolution: implemented
Status: merge_readyclosed

Merged!

Note: See TracTickets for help on using tickets.