Differential D21178

Fix an invalid index access for synthetic lint inline comments from Harbormaster
ClosedPublic
Actions

Authored by epriestley on Apr 27 2020, 9:18 PM.

Tags

None

Referenced Files

	F18734269: D21178.id.diff
	Tue, Sep 30, 10:51 PM

	F18606320: D21178.diff
	Sep 13 2025, 10:05 PM

	F18600611: D21178.id50432.diff
	Sep 13 2025, 10:07 AM

	F18563327: D21178.diff
	Sep 9 2025, 7:00 AM

	F18519465: D21178.diff
	Sep 5 2025, 5:13 PM

	F18228863: D21178.id.diff
	Aug 20 2025, 8:16 AM

	F18222715: D21178.diff
	Aug 19 2025, 9:34 PM

	F18106752: D21178.id50433.diff
	Aug 10 2025, 11:19 PM

Subscribers

None

Details

Reviewers: None
Maniphest Tasks: T13524: "Undefined index" exception in ChangesetParser while viewing diffs
Commits: rPe17c49699309: (stable) Fix an invalid index access for synthetic lint inline comments from…
rPaa20faeaa5d1: Fix an invalid index access for synthetic lint inline comments from Harbormaster

Summary

Ref T13524. If a Harbormaster lint message has no line number (which is permitted), we try to access an invalid index here. This is an exception after D21044.

Treat comments with no line number as unchanged. These comments do not have "ghost" behavior and do not port across diffs.

Test Plan

Used "harbormaster.sendmessage" to submit lint with no line number on a changeset.
Viewed changeset.
- Before patch: "Undefined index: <null>" error.
- After patch: Clean changeset with lint message.

Screen Shot 2020-04-27 at 2.17.38 PM.png (236×856 px, 18 KB)

Diff Detail

Repository

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

epriestley created this revision.Apr 27 2020, 9:18 PM

Harbormaster completed remote builds in B24251: Diff 50432.Apr 27 2020, 9:19 PM

This revision was not accepted when it landed; it landed in state Needs Review.Apr 27 2020, 9:19 PM

epriestley requested review of this revision.

Closed by commit rPaa20faeaa5d1: Fix an invalid index access for synthetic lint inline comments from Harbormaster (authored by epriestley). · Explain Why

This revision was automatically updated to reflect the committed changes.

epriestley added a commit: rPaa20faeaa5d1: Fix an invalid index access for synthetic lint inline comments from Harbormaster.

epriestley added a commit: rPe17c49699309: (stable) Fix an invalid index access for synthetic lint inline comments from….

epriestley mentioned this in D18302: Fix "undefined index" error in Differential changeset parser.Mar 17 2021, 7:31 PM

epriestley mentioned this in T8591: Undefined index in changeset parser.

Revision Contents
Changeset List

Path

Size

src/

applications/

differential/

parser/

DifferentialChangesetParser.php

12 lines

Diff 50433

src/applications/differential/parser/DifferentialChangesetParser.php

Loading...