diff options
author | Jakub Narebski <jnareb@gmail.com> | 2006-09-26 01:57:02 +0200 |
---|---|---|
committer | Junio C Hamano <junkio@cox.net> | 2006-09-27 00:41:35 -0700 |
commit | 24d0693a68008baacd827b1345c957e871488596 (patch) | |
tree | 2976f3d666212dece1e35f00c729194ee1f0ea4b /builtin-diff.c | |
parent | gitweb: Use "return" instead of "return undef" for some subs (diff) | |
download | tgif-24d0693a68008baacd827b1345c957e871488596.tar.xz |
gitweb: Split validate_input into validate_pathname and validate_refname
Split validate_input subroutine into validate_pathname which is used
for $project, $file_name and $file_parent parameters, and
validate_refname which is used for $hash, $hash_base, $hash_parent and
$hash_parent_base parameters. Reintroduce validation of $file_name
and $file_parent parameters, removed in a2f3db2f
validate_pathname in addition to what validate_input did checks also
for doubled slashes and NUL character. It does not check if input is
textual hash, and does not check if all characters are from the
following set: [a-zA-Z0-9_\x80-\xff\ \t\.\/\-\+\#\~\%].
validate_refname first check if the input is textual hash, then checks
if it is valid pathname, then checks for invalid characters (according
to git-check-ref-format manpage). It does not check if all charactes
are from the [a-zA-Z0-9_\x80-\xff\ \t\.\/\-\+\#\~\%] set.
We do not have to validate pathnames we got from git.
Signed-off-by: Jakub Narebski <jnareb@gmail.com>
Signed-off-by: Junio C Hamano <junkio@cox.net>
Diffstat (limited to 'builtin-diff.c')
0 files changed, 0 insertions, 0 deletions