On Mar 28, 2012 4:23 AM, "Daniel Shahaf" <d...@daniel.shahaf.name> wrote: > > gst...@apache.org wrote on Wed, Mar 28, 2012 at 01:51:54 -0000: > > Author: gstein > > Date: Wed Mar 28 01:51:54 2012 > > New Revision: 1306111 > > > > URL: http://svn.apache.org/viewvc?rev=1306111&view=rev > > Log: > > Fix a problem in the svn protocol's get_dir() functionality. The tuple > > specification allowed for a NULL created-date, but the client would > > segfault with that input (via svn_time_from_string). > > > > Backport?
For client resilience, sure. But very, very low footprint/issue. IMO, not a release driver, but something to get caught up into a release. > > > - svn_dirent_t entry; > > - memset(&entry, 0, sizeof(entry)); > > + /* The fields in the entry tuple. */ > > + svn_node_kind_t entry_kind = svn_node_none; > > + svn_filesize_t entry_size = 0; > > + svn_boolean_t has_props = FALSE; > > + svn_revnum_t created_rev = 0; /* ### SVN_INVALID_REVNUM */ > > + const char *cdate = NULL; > > + const char *last_author = NULL; > > Unrelated change. Right. Agreed. Only client.c is interesting for a backport. Cheers, -g
