HomePhabricator

Remove accidental sprintf injection in error reporting

Description

Remove accidental sprintf injection in error reporting

Summary:
STDERR output with %s in it could cause:

ERROR 2: fprintf(): Too few arguments at [/usr/local/arcanist/src/workflow/ArcanistFeatureWorkflow.php:170]

Test Plan: Untested.

Reviewers: Blessed Reviewers, epriestley

Reviewed By: Blessed Reviewers, epriestley

Subscribers: Korvin, epriestley

Differential Revision: https://secure.phabricator.com/D19261

Details

Provenance
alexmvAuthored on Mar 26 2018, 5:36 PM
alexmvPushed on Mar 26 2018, 8:56 PM
Reviewer
Blessed Reviewers
Differential Revision
D19261: Remove accidental sprintf injection in error reporting
Parents
rARCdcd7ef66d0e4: Add support for Go 1.10 (cached) output
Branches
Unknown
Tags
Unknown
Build Status
Buildable 19928
Build 27014: Run Core Tests