2005-04-29 16:22:13

by Coywolf Qi Hunt

[permalink] [raw]
Subject: [patch] lxr-0.3.1 handle `_' fix


Hello,

lxr-0.3.1 can not handle '_' in the Architecture variable.

Like <http://sosdg.org/~coywolf/lxr/source/include/asm-x86_64/?a=x86_64>
is wrongly shown as <http://sosdg.org/~coywolf/lxr/source/include/asm-/?a=x86_64>

This problem was brought in by the file exposure security fix some time ago iirc.

Coywolf

diff -pu lxr/http/lib/LXR/Config.pm.orig lxr/http/lib/LXR/Config.pm
--- lxr/http/lib/LXR/Config.pm.orig 2005-04-29 23:35:26.000000000 +0800
+++ lxr/http/lib/LXR/Config.pm 2005-04-29 23:37:28.000000000 +0800
@@ -156,7 +156,7 @@ sub varrange {
sub varexpand {
my ($self, $exp) = @_;
$exp =~ s{\$\{?(\w+)\}?}{
- $self->{variable}->{$1} =~ /^([a-zA-Z0-9\.\-]*)$/ ? $1 : ''
+ $self->{variable}->{$1} =~ /^([a-zA-Z0-9\.\-_]*)$/ ? $1 : ''
}ge;
return($exp);
}