Page MenuHomePhabricator

Remove accidental sprintf injection in error reporting
ClosedPublic

Authored by alexmv on Mar 26 2018, 5:38 PM.
Tags
None
Referenced Files
Unknown Object (File)
Fri, Mar 1, 3:45 AM
Unknown Object (File)
Thu, Feb 29, 3:42 PM
Unknown Object (File)
Feb 21 2024, 11:34 AM
Unknown Object (File)
Feb 18 2024, 5:50 PM
Unknown Object (File)
Feb 7 2024, 10:48 PM
Unknown Object (File)
Feb 3 2024, 6:36 PM
Unknown Object (File)
Feb 3 2024, 6:36 PM
Unknown Object (File)
Jan 27 2024, 8:32 PM
Subscribers

Details

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.

Diff Detail

Repository
rARC Arcanist
Branch
sprintf-injection
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 19927
Build 27013: Run Core Tests
Build 27012: arc lint + arc unit

Event Timeline

I'm getting permanent build failures on this diff, which look unrelated to the diff itself?

I'll take a look at the tests, definitely doesn't look related.

This revision is now accepted and ready to land.Mar 26 2018, 8:34 PM

The test failures are just T10093, i.e. arc didn't push the changes to the staging environment. Most common reason is that you're working off a local fork, not the One True Upstream Repository?

Yeah, that's prbably exactly it -- we push staging refs to a different namespace to not clutter the standard tag namespace, which is likely the problem.