]> git.ipfire.org Git - thirdparty/git.git/commit - upload-pack.c
upload-pack: disable object filtering when disabled by config
authorJonathan Nieder <jrnieder@gmail.com>
Wed, 28 Mar 2018 20:33:03 +0000 (13:33 -0700)
committerJunio C Hamano <gitster@pobox.com>
Thu, 29 Mar 2018 22:39:31 +0000 (15:39 -0700)
commitc7620bd0f35dddf8b8519da6fbf97014f46d0710
treef13a1b9f57d7530ec55e33b604b895ac74c35e2f
parent9f242a13366d4ca15358780d3b32aa1076dd7f49
upload-pack: disable object filtering when disabled by config

When upload-pack gained partial clone support (v2.17.0-rc0~132^2~12,
2017-12-08), it was guarded by the uploadpack.allowFilter config item
to allow server operators to control when they start supporting it.

That config item didn't go far enough, though: it controls whether the
'filter' capability is advertised, but if a (custom) client ignores
the capability advertisement and passes a filter specification anyway,
the server would handle that despite allowFilter being false.

This is particularly significant if a security bug is discovered in
this new experimental partial clone code.  Installations without
uploadpack.allowFilter ought not to be affected since they don't
intend to support partial clone, but they would be swept up into being
vulnerable.

Simplify and limit the attack surface by making uploadpack.allowFilter
disable the feature, not just the advertisement of it.

Signed-off-by: Jonathan Nieder <jrnieder@gmail.com>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
Documentation/config.txt
upload-pack.c