You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@httpd.apache.org by Rob Hartill <ha...@ooo.lanl.gov> on 1996/02/13 07:14:45 UTC

vote update

Please save pixels by only voting from now on if it will have
any affect on the patch, e.g. a veto or +1 to give a patch 3 votes.
  (eh?, well for example 56.alias_userdir doesn't need any more votes, so
  don't waste time typing them)


AK  Alexei Kosut
AM  Aram Mirzadeh
RH  Rob Hartill
RT  Randy Terbush
BB  Brian Behlendorf
CM  Chuck Murcko


Patch #               AK  AM  RH  RT  BB  CM            Accept/Veto

23a.mmap              -1   1  -1*d 0   1   1            Veto
23b.mmap                       1                        
56.alias_userdir       1   1   1   1   1   1            A
61a.preserve_redirect  1   1   1   1   1   1            A
62a.escape_html        1   1   1   1   1   1            A
65b.new-make           1  -1  -1  -1   1  -1            Veto
66.htaccess-cache      1   1   1   1   1   1            A
68b.strftime           1   1   1   1   1   1            A
70c.LynxOS_port        1   1   1   1   1   0            A
72.mod_imap_overhaul   1   1   0  -1  -1  -1            Veto
72a.mod_imap_overhaul  1  -1   0  -1  -1  -1            Veto
73a.mod_actions        1   1   0   1   1   1            A
74.icons               1   1   1   1   1   1            A
75.icons               1   1   1   1   1   1            A
76a.posix_wait4proces  1   1   1   1   1   1            A
77.user_name           1   1   1   1   1   1            A
78.preserve_redirect   1   1   1   1   1   1            A
82.setsockopt_next     1   1  -1   1   0   0            Veto
82b.setsockopt_next            1                        
83.timefmt             0   1  -1  -1  -1  -1            Veto
84b.timeout            1   1   1   1   1   1            A
85.lost_conn           1   1   1   1   1   1            A
86a.scoreboard_into_l  1   1   1   1   1   1            A
87.rlimit_warning      1   1   1   1   1   1            A
88.cookie_pstrcat      1   1   1   1   1   1            A
89.minSpare            1   1   1   1   1   1            A
90g.keepalive          1   1   1   1   1   1            A
91.config_dns          1   1   1   1   1   1            A
92.alias_htaccess      1   1   1   0   1   1            A
93.opt_all_mv          1   1   1   1  -1   1            Veto
94.httpd_monitor       0   1   0  -1   0   1            Veto
95.mod_neg.useragent   0   1   1   1  -1   1            Veto
96.hpux.RLIMIT         1   1   1   1   1   0            A
97.proxy-02            1*a 1   1   1   1   0            A
98.errorlog            1   1   1   1   1   1            A
99.bind                1*b 0   0   1   1   1*c          A
100d.os2_port                  1           1
101.add_strerror_to    1       1   1       0            A
102.proxy_hpux.patch   1      -1           0            Veto  (obsolete)
apache-msql-demo       0   1   1   0   1   0            A    ! for Contrib/
logresolve.c           0   1   1   0   1   0            A
mod_alias_map.c        0   1   1   1       0            A
mod_auth_anon.c        0   1   1   1   1   0            A
mod_auth_db.c          0   1   1   1   1   0            A
mod_auth_msql.c        0   1   1   1   1   0            A
mod_cern_meta.c        1   1   1   1   1   1            A
mod_env.c              1   1   1   1   1   1            A


(*a   -1 unless SEGV bug is fixed with 102)

(*b Works fine on HP-UX. BTW, I've figured out how to make 82 (with or without
the #ifdef NEXT) and 99 co-exist (thanks, Randy). Assuming this is done,
change my -1 of 99 to a +1.)
(*c changes to +1 if it works with 82 - works fine
                             This needs to be remade anyway, the offsets
                                get huge by this one, and break plain ole
                                patch.)
                                
(*d in favour of the replacment)



Re: vote update

Posted by Alexei Kosut <ak...@nueva.pvt.k12.ca.us>.
On Mon, 12 Feb 1996, Alexei Kosut wrote:

> > 97.proxy-02            1*a 1   1   1   1   0            A
> 
> This was replaced with 97.proxy-03 (which does work). I'll -1 this, and 
> +1 that.

Wait a minute...

Just got a chance to look more closely at 97.proxy-03 on my wonderful
HP-UX which likes nothing (except it happily runs with things that make
SunOSes very unhappy, witness the mod_actions seg fault of a while back). 
Anyhow, it's SIGSEGVing again. Which I find odd, because except for the
what's in patch 102 (which it worked fine with last time), nothing in
mod_proxy.c has changed. David said he made one other change, maybe this
is causing it. Or maybe I've just gone crazy. 

I can't figure out what's causing it, but a gdb "where" on the resulting
core file gives me (if this is any use to anyone): 

#0  0x802d3ff8 in kill ()
#1  0x802b4698 in raise ()
#2  0x802b4248 in abort ()
#3  0xc658 in seg_fault () at http_main.c:660
#4  <signal handler called>
#5  0x802baf60 in _time ()
#6  0x24bd0 in http_handler (r=0x40010718, c=0x46464646,
    url=0x9000 "rmissions deny server access",
    proxyhost=0x4001c6d8 "@\001\0068", proxyport=0) at mod_proxy.c:2600
#7  0x237ac in proxy_handler (r=0x40010718) at mod_proxy.c:1983
#8  0xf19c in invoke_handler (r=0x40010718) at http_config.c:287
#9  0x10c4c in process_request_internal (r=0x40010718) at http_request.c:662
#10 0x10c78 in process_request (r=0x40010718) at http_request.c:668
#11 0xcd70 in child_main (child_num_arg=39692) at http_main.c:940
#12 0xce94 in make_child (server_conf=0x4000a620, child_num=0)
    at http_main.c:977
#13 0xd2e0 in standalone_main (argc=39692, argv=0x40010718) at
    http_main.c:1104
#14 0xd658 in main (argc=2, argv=0x7b033268) at http_main.c:1207

--/ Alexei Kosut <ak...@nueva.pvt.k12.ca.us> /--------/ Lefler on IRC
----------------------------/ <http://www.nueva.pvt.k12.ca.us/~akosut/>
The viewpoints expressed above are entirely false, and in no way
represent Alexei Kosut nor any other person or entity. /--------------



Re: vote update

Posted by Alexei Kosut <ak...@nueva.pvt.k12.ca.us>.
On Mon, 12 Feb 1996, Rob Hartill wrote:

> 82b.setsockopt_next            1                        

+1. But it needs a version of 99 to be applied first, so a conditional -1 
    on this if 99 isn't passed too. (if it isn't, plain ol' 82 will work)

> 97.proxy-02            1*a 1   1   1   1   0            A

This was replaced with 97.proxy-03 (which does work). I'll -1 this, and 
+1 that.

> 99.bind                1*b 0   0   1   1   1*c          A

How about we just toss this and put in 99b.bind, which I'll +1, since it 
works, and patches cleanly.

> 100d.os2_port                  1           1

+1. (it patches cleanly, at least, abliet screwing up Configuration, 
which can be fixed. But I can't test it on OS/2)

> 101.add_strerror_to    1       1   1       0            A

Something up there (23, I think) makes it so part of this patch has to
applied manually. But it's only one line, easily fixable, and my +1 stays. 

> 102.proxy_hpux.patch   1      -1           0            Veto  (obsolete)

Can just be removed and superceded by 97.proxy-03.

--/ Alexei Kosut <ak...@nueva.pvt.k12.ca.us> /--------/ Lefler on IRC
----------------------------/ <http://www.nueva.pvt.k12.ca.us/~akosut/>
The viewpoints expressed above are entirely false, and in no way
represent Alexei Kosut nor any other person or entity. /--------------