[svn.haxx.se] · SVN Dev · SVN Users · SVN Org · TSVN Dev · TSVN Users · Subclipse Dev · Subclipse Users · this month's index

Re: backport.pl still misbehaving

From: Daniel Shahaf <d.s_at_daniel.shahaf.name>
Date: Thu, 24 Apr 2014 16:24:27 +0000

Philip Martin wrote on Thu, Apr 24, 2014 at 11:29:34 +0100:
> svn-role_at_apache.org writes:
>
> > Author: svn-role
> > Date: Thu Apr 24 04:01:09 2014
> > New Revision: 1589570
> >
> > URL: http://svn.apache.org/r1589570
> > Log:
> > Merge r1587968 from trunk:
> >
> > * r1587968
> > Maintain data[len]=0 invariant for svn_string_t.
> > Justification:
> > Code may rely on null byte being present.
> > Notes:
> > Triggers a read beyond allocated memory on trunk but no known
> > misbehaviour on 1.8.
> > Votes:
> > +1: philip, rhuijben, stefan2
> >
> > Modified:
> > subversion/branches/1.8.x/ (props changed)
> > subversion/branches/1.8.x/STATUS
> > subversion/branches/1.8.x/subversion/libsvn_delta/svndiff.c
> >
> > Propchange: subversion/branches/1.8.x/
> > ------------------------------------------------------------------------------
> > Merged /subversion/trunk:r1587968
> >
> > Modified: subversion/branches/1.8.x/STATUS
> > URL: http://svn.apache.org/viewvc/subversion/branches/1.8.x/STATUS?rev=1589570&r1=1589569&r2=1589570&view=diff
> > ==============================================================================
> > --- subversion/branches/1.8.x/STATUS (original)
> > +++ subversion/branches/1.8.x/STATUS Thu Apr 24 04:01:09 2014
> > @@ -303,15 +303,6 @@ Approved changes:
> > Votes:
> > +1: philip, rhuijben, stefan2
> >
> > - * 1585499
> > - Build with apr-0.9.x.
> > - Justification:
> > - 1.8 claims 0.9 support but it doesn't work.
> > - Branch:
> > - ^/subversion/branches/1.8.x-apr-0.9
> > - Votes:
> > - +1: philip, rhuijben, stefan2
> > -
> > * r1581810, 1584342
> > Fix issue 4412, locks lost from working copy with pre-1.6.17 servers.
> > Justification:
> >
>
> The log message matches the code change but the STATUS change is wrong.

For the records, these commits were done by:
    % ssh svn-qavm.apache.org 'tail -2 ~svnsvn/live-version.log'
    20140424:
    1569237
and that revision was supposed to be predating the "wrong STATUS entry
deleted" bug.

I've gone ahead and disabled svn-role by commenting out the crontab
entry, to avoid any further breakage. How merges are to be done in the
meantime is up to the RM.

I'll also note that the cron job generated an error tonight (the output
was mailed to whoever is on the svnsvn_at_svn-qavm alias in /etc/aliases).
Received on 2014-04-24 18:25:06 CEST

This is an archived mail posted to the Subversion Dev mailing list.

This site is subject to the Apache Privacy Policy and the Apache Public Forum Archive Policy.