Paths

Table of Contentst

tarfs: Limit file name length
ClosedPublic
Actions

Authored by des on Tue, Oct 20, 2:42 PM.

Project Tags

None

Referenced Files

	F177490828: D60903.1794552991.diff
	Thu, Nov 12, 6:56 AM

	F177486663: D60903.1794547336.diff
	Thu, Nov 12, 5:22 AM

Subscribers

imp

Details

Reviewers

emaste
markj

Commits

rGbee1ce4727ed: tarfs: Limit file name length
rG6418155c47fd: tarfs: Limit file name length
rGa38033234692: tarfs: Limit file name length

Summary

When indexing the tarball on mount, fail if a file or directory name
exceeds MAXNAMLEN.

Reported by: n0ps1ed
Reported by: Franz Bettag <franz@bett.ag>
MFC after: 1 week

Diff Detail

Repository

rG FreeBSD src repository

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

des created this revision.Tue, Oct 20, 2:42 PM

Herald added a subscriber: imp. · View Herald TranscriptTue, Oct 20, 2:42 PM

des requested review of this revision.Tue, Oct 20, 2:42 PM

des added a parent revision: D60902: tftpd: Keep proper track of header length.

Harbormaster completed remote builds in B78416: Diff 190396.Tue, Oct 20, 2:42 PM

des added a child revision: D60904: tarfs: Fix length calculation for file read.Tue, Oct 20, 2:42 PM

emaste accepted this revision.Tue, Oct 20, 3:08 PM

This revision is now accepted and ready to land.Tue, Oct 20, 3:08 PM

Use NAME_MAX instead of MAXNAMLEN; they are equivalent, but the latter is not already in scope.

This revision now requires review to proceed.Tue, Oct 20, 3:55 PM

Harbormaster completed remote builds in B78421: Diff 190403.Tue, Oct 20, 3:55 PM

markj accepted this revision.Thu, Oct 29, 8:08 PM

This revision is now accepted and ready to land.Thu, Oct 29, 8:08 PM

emaste accepted this revision.Thu, Oct 29, 8:15 PM

This was also reported by Franz Bettag <franz@bett.ag>

des edited the summary of this revision. (Show Details)Fri, Oct 30, 11:22 AM

Closed by commit rGa38033234692: tarfs: Limit file name length. · Explain WhyMon, Nov 2, 12:38 PM

This revision was automatically updated to reflect the committed changes.

des added a commit: rGa38033234692: tarfs: Limit file name length.

des added a commit: rG6418155c47fd: tarfs: Limit file name length.Thu, Nov 5, 7:33 PM

des added a commit: rGbee1ce4727ed: tarfs: Limit file name length.

Revision Contents
Changeset List

Path

Size

sys/

fs/

tarfs/

tarfs_subr.c

2 lines

Diff 191291

View Options

tarfs: Limit file name lengthClosedPublicActions