Paths

Table of Contentst

Differential D19882

Fix construction of two new qsprintf() exceptions
ClosedPublic
Actions

Authored by epriestley on Dec 13 2018, 7:01 PM.

Tags

None

Referenced Files

	F15507252: D19882.id47481.diff
	Tue, Apr 15, 5:54 PM

	F15502719: D19882.id.diff
	Mon, Apr 14, 3:31 AM

	F15500260: D19882.diff
	Sun, Apr 13, 5:08 PM

	F15482606: D19882.id47478.diff
	Wed, Apr 9, 6:32 AM

	F15477900: D19882.id47481.diff
	Mon, Apr 7, 6:41 PM

	F15444477: D19882.diff
	Thu, Mar 27, 9:46 AM

	F15381660: D19882.diff
	Mar 14 2025, 8:47 AM

	F15376531: D19882.id47481.diff
	Mar 13 2025, 4:18 AM

View All Files

Subscribers

None

Details

Reviewers

amckinley

Maniphest Tasks

T13217: Upgrading: Hardening of qsprintf()
T13222: 2018 Week 48-51 Bonus Content

Commits

rPHUa537ba03c994: (stable) Promote 2018 Week 50
rPHUcad1985726c9: Fix construction of two new qsprintf() exceptions

Summary

Ref T13222. Ref T13217. These exception signatures aren't quite right, they should take $query as the first parameter.

Test Plan

Misused %P. Before: error about exception construction. After: useful exception.

Diff Detail

Repository

rPHU libphutil

Branch

mfa3

Lint

Lint Passed

Unit

Tests Passed

Build Status

Buildable 21329
Build 29024: Run Core Tests
Build 29023: arc lint + arc unit

Event Timeline

epriestley created this revision.Dec 13 2018, 7:01 PM

Harbormaster completed remote builds in B21329: Diff 47478.Dec 13 2018, 7:02 PM

epriestley requested review of this revision.Dec 13 2018, 7:02 PM

amckinley accepted this revision.Dec 13 2018, 7:49 PM

This revision is now accepted and ready to land.Dec 13 2018, 7:49 PM

Closed by commit rPHUcad1985726c9: Fix construction of two new qsprintf() exceptions (authored by epriestley). · Explain WhyDec 13 2018, 8:22 PM

This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Path

Size

src/

xsprintf/

qsprintf.php

2 lines

Diff 47478

View Options

Fix construction of two new qsprintf() exceptionsClosedPublicActions