httpd-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Torsten Foertsch <torsten.foert...@gmx.net>
Subject The right way to report problems (was: uninitialized variable in ap_directory_walk)
Date Thu, 15 Mar 2007 19:20:05 GMT
Hi,

almost a week ago I have reported an uninitialized variable in 
server/request.c. I have also filed a bug 
(http://issues.apache.org/bugzilla/show_bug.cgi?id=41829).

The bug is simple, the patch is simple. Why haven't I got a single reply to my 
mail? The bug is also still marked as new. What is the right way to report 
problems?

But maybe I should explain the problem a bit. With mod_perl you can add 
configuration statements by $r->add_config(...). This can also be done in a 
maptostorage handler which comes handy because it allows you to set some 
initial values for AllowOverride or Options which are evaluated by the core 
maptostorage handler. So, with httpd 2.0.x this has worked without problem. 
With 2.2 the new override_opts were introduced. ap_directory_walk maintains 
its own copy of these flags along with the current override options in a 
structure named opts. The members of this structure are initialized from the 
requests per_dir_config. But somehow the new override_opts had been 
forgotten. By cause the override_opts member was in my installation always 0 
thus preventing to set any other Options via .htaccess.

Torsten

On Sunday 11 March 2007 18:01, Torsten Foertsch wrote:
> this bug is alive since 2.2.0. The patch is against trunk.
>
> --- server/request.c~   2007-03-11 17:20:25.000000000 +0100
> +++ server/request.c    2007-03-11 17:50:01.000000000 +0100
> @@ -631,6 +631,7 @@
>          opts.add = this_dir->opts_add;
>          opts.remove = this_dir->opts_remove;
>          opts.override = this_dir->override;
> +        opts.override_opts = this_dir->override_opts;
>
>          /* Set aside path_info to merge back onto path_info later.
>           * If r->filename is a directory, we must remerge the path_info,

Mime
View raw message