Paths

Table of Contentst

Differential D19451

Parse unusual Subversion protocol frames which contain extra whitespace
ClosedPublic
Actions

Authored by epriestley on May 15 2018, 10:58 PM.

Tags

None

Referenced Files

	F14760187: D19451.id46532.diff
	Wed, Jan 22, 4:59 PM

	Unknown Object (File)
	Tue, Jan 21, 3:55 PM

	Unknown Object (File)
	Tue, Jan 21, 9:25 AM

	Unknown Object (File)
	Dec 14 2024, 4:56 PM

	Unknown Object (File)
	Dec 6 2024, 6:10 AM

	Unknown Object (File)
	Dec 4 2024, 1:00 PM

	Unknown Object (File)
	Nov 28 2024, 9:54 PM

	Unknown Object (File)
	Nov 27 2024, 3:12 AM

Subscribers

None

Details

Reviewers

amckinley
asherkin

Maniphest Tasks

T13140: Phabricator fails to parse unusual protocol requests emitted by Subversion

Commits

rP3544620209cf: Parse unusual Subversion protocol frames which contain extra whitespace

Summary

Fixes T13140. See PHI660.

Recent versions of Subversion can send a (get-file true false false ) protocol frame with extra space between "false" and "false". This is allowed by the protocol spec, but never normally happens, and we do not parse it correctly.

Instead, parse it correctly.

Test Plan

Added unit tests.
Ran svn proplist svn+ssh://.../diffusion/X/file.c under SVN 1.10 before and after the change.
- Before: indefinite hang.
- After: completed in finite time.

Diff Detail

Repository

Branch

svn1

Lint

Lint Passed

Unit

Tests Passed

Build Status

Buildable 20336
Build 27617: Run Core Tests
Build 27616: arc lint + arc unit

Event Timeline

epriestley created this revision.May 15 2018, 10:58 PM

Harbormaster completed remote builds in B20336: Diff 46528.May 15 2018, 10:59 PM

epriestley requested review of this revision.May 15 2018, 10:59 PM

asherkin accepted this revision.May 16 2018, 2:29 PM

This revision is now accepted and ready to land.May 16 2018, 2:29 PM

amckinley accepted this revision.May 16 2018, 5:16 PM

Closed by commit rP3544620209cf: Parse unusual Subversion protocol frames which contain extra whitespace (authored by epriestley). · Explain WhyMay 17 2018, 12:12 AM

This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Path

Size

src/

applications/

diffusion/

protocol/

DiffusionSubversionWireProtocol.php

27 lines

__tests__/

DiffusionSubversionWireProtocolTestCase.php

59 lines

Diff 46528

src/applications/diffusion/protocol/DiffusionSubversionWireProtocol.php

Loading...

src/applications/diffusion/protocol/tests/DiffusionSubversionWireProtocolTestCase.php

Loading...