Differential D19872

Fix a stray qsprintf() in the Herald rules engine when recording rule application to objects
ClosedPublic
Actions

Authored by epriestley on Dec 12 2018, 6:58 PM.

Tags

None

Referenced Files

	Unknown Object (File)
	Fri, Apr 19, 8:09 PM

	Unknown Object (File)
	Mar 19 2024, 11:00 PM

	Unknown Object (File)
	Mar 10 2024, 8:24 PM

	Unknown Object (File)
	Feb 19 2024, 6:24 PM

	Unknown Object (File)
	Feb 18 2024, 4:36 PM

	Unknown Object (File)
	Feb 3 2024, 8:12 PM

	Unknown Object (File)
	Jan 20 2024, 4:34 PM

	Unknown Object (File)
	Jan 16 2024, 5:13 PM

View All Files

Subscribers

None

Details

Reviewers

amckinley

Maniphest Tasks

T13217: Upgrading: Hardening of qsprintf()

Commits

rP2814d340367c: Fix a stray qsprintf() in the Herald rules engine when recording rule…

Summary

Ref T13217. See PHI1006.

Test Plan

Touched an object with associated Herald rules.

Diff Detail

Repository

rP Phabricator

Branch

query2

Lint

Lint Passed

Unit

Tests Passed

Build Status

Buildable 21306
Build 28988: Run Core Tests
Build 28987: arc lint + arc unit

Event Timeline

epriestley created this revision.Dec 12 2018, 6:58 PM

Harbormaster completed remote builds in B21306: Diff 47453.Dec 12 2018, 7:00 PM

epriestley requested review of this revision.Dec 12 2018, 7:00 PM

amckinley accepted this revision.Dec 12 2018, 7:15 PM

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

Closed by commit rP2814d340367c: Fix a stray qsprintf() in the Herald rules engine when recording rule… (authored by epriestley). · Explain WhyDec 12 2018, 7:31 PM

This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Path

Size

src/

applications/

herald/

engine/

HeraldEngine.php

4 lines

Diff 47453

View Options

src/applications/herald/engine/HeraldEngine.php

Fix a stray qsprintf() in the Herald rules engine when recording rule application to objectsClosedPublicActions