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

Re: svn commit: r1561570 - /subversion/trunk/subversion/libsvn_ra_svn/marshal.c

From: Stefan Fuhrmann <stefan.fuhrmann_at_wandisco.com>
Date: Mon, 27 Jan 2014 01:28:29 +0100

On Mon, Jan 27, 2014 at 12:48 AM, Bert Huijben <bert_at_qqmail.nl> wrote:

>
>
> > -----Original Message-----
> > From: stefan2_at_apache.org [mailto:stefan2_at_apache.org]
> > Sent: maandag 27 januari 2014 00:00
> > To: commits_at_subversion.apache.org
> > Subject: svn commit: r1561570 -
> > /subversion/trunk/subversion/libsvn_ra_svn/marshal.c
> >
> > Author: stefan2
> > Date: Sun Jan 26 22:59:29 2014
> > New Revision: 1561570
> >
> > URL: http://svn.apache.org/r1561570
> > Log:
> > Remove false protocol limit checks from the ra_svn marshaller.
> >
> > * subversion/libsvn_ra_svn/marshal.c
> > (svn_ra_svn__read_data_log_changed_entry): Remove upper limit
> > check for variable length elements as they may be extended by
> > later releases.
>
> I don't think we can change these entries without a time machine if these
> limits were coded in previously released versions?
>
> Were these checks added for 1.9?
>

Yes, they had and those checks were too strict. The protocol
allows for adding an unlimited number of optional elements.

-- Stefan^2.
Received on 2014-01-27 01:29:02 CET

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.