Page MenuHomeFreeBSD
Differential D44599

tarfs: Support paths that spill into exthdrs.
ClosedPublic
Actions

Authored by des on Tue, Apr 2, 8:15 PM.
Tags
None
Referenced Files
Unknown Object (File)
Sat, Apr 27, 3:11 AM
Unknown Object (File)
Fri, Apr 26, 4:40 AM
Unknown Object (File)
Sat, Apr 6, 8:23 PM
Unknown Object (File)
Fri, Apr 5, 6:33 PM
Unknown Object (File)
Wed, Apr 3, 9:59 AM
Subscribers
imp
kevans
nonesuch_longcount.org

Details

Reviewers
sjg
kevans
Group Reviewers
Klara
Commits
rG59c3e7a18c7c: tarfs: Support paths that spill into exthdrs.
rGb1fd95c9e247: tarfs: Support paths that spill into exthdrs.
Summary

MFC after: 3 days
Sponsored by: Juniper Networks, Inc.
Sponsored by: Klara, Inc.

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

des created this revision.Tue, Apr 2, 8:15 PM
Herald added a subscriber: imp. · View Herald TranscriptTue, Apr 2, 8:15 PM
des requested review of this revision.Tue, Apr 2, 8:15 PM
Harbormaster completed remote builds in B56875: Diff 136449.Tue, Apr 2, 8:15 PM
des added a child revision: D44600: tarfs: Ignore global extended headers..Tue, Apr 2, 8:15 PM
des added a subscriber: nonesuch_longcount.org.Tue, Apr 2, 8:26 PM
kevans accepted this revision.Wed, Apr 3, 4:48 AM
kevans added a subscriber: kevans.
kevans added inline comments.
sys/fs/tarfs/tarfs_vfsops.c
619

Is there any value in complaining if we somehow see both a path and GNU.sparse.name exthdr?

This revision is now accepted and ready to land.Wed, Apr 3, 4:48 AM
des marked an inline comment as done.Wed, Apr 3, 9:14 AM
des added inline comments.
sys/fs/tarfs/tarfs_vfsops.c
619

I don't think so. I don't see how it can be exploited. We'll just use whichever came last, we won't leak memory or take a wrong turn somewhere.

des marked an inline comment as done.Wed, Apr 3, 9:14 AM
Closed by commit rGb1fd95c9e247: tarfs: Support paths that spill into exthdrs. (authored by des). · Explain WhyWed, Apr 3, 10:00 AM
This revision was automatically updated to reflect the committed changes.
des added a commit: rGb1fd95c9e247: tarfs: Support paths that spill into exthdrs..
des added a commit: rG59c3e7a18c7c: tarfs: Support paths that spill into exthdrs..Mon, Apr 8, 10:11 AM

Revision Contents
Changeset List

PathSize
sys/
fs/
tarfs/
7 lines
tests/
sys/
fs/
tarfs/
50 lines

Diff 136470

View Options

sys/fs/tarfs/tarfs_vfsops.c

Loading...
View Options

tests/sys/fs/tarfs/tarfs_test.sh

Loading...