[PATCH v3 0/9] http: end urls with slash
- From
Tay Ray Chuan <rctay89@gmail.com>
- Date
- Nov 25, 2010, 08:21 UTC
- Message-ID
- <1290673270-4284-1-git-send-email-rctay89@gmail.com>
- In-Reply-To
- <1290433298-6000-1-git-send-email-rctay89@gmail.com>
This patch series contains Gabriel's fix, as well as some related work I did in the area while looking through the code paths.
Changed from v2: - new patches (#2 and #9) that reworks http-fetch url slashing code, and test that this doesn't break anything. - patch #4: remove pointer check before running free(), as pointed out by Johnathan.
All other patches remain unchanged.
Contents: [1/9] t5550-http-fetch: add missing '&&' [2/9] t5550-http-fetch: add test for http-fetch [3/9] shift end_url_with_slash() from http.[ch] to url.[ch] [4/9] url: add str wrapper for end_url_with_slash() [5/9] http-backend: use end_url_with_slash() [6/9] http-push: Normalise directory names when pushing to some WebDAV servers [7/9] http-push: check path length before using it [8/9] http-push: add trailing slash at arg-parse time, instead of later on [9/9] http-fetch: rework url handling
Makefile | 2 +- http-backend.c | 4 +--- http-fetch.c | 16 +++++----------- http-push.c | 31 ++++++++++++++++--------------- http.c | 8 +------- http.h | 2 +- t/t5550-http-fetch.sh | 16 ++++++++++++++-- url.c | 14 ++++++++++++++ url.h | 3 +++ 9 files changed, 56 insertions(+), 40 deletions(-)
-- 1.7.3.2.495.gc7b3f