On Fri, Dec 06, 2019 at 03:05:38PM +0300, Dmitry V. Levin wrote:
> On Fri, Dec 06, 2019 at 02:15:54PM +0300, Aleksei Nikiforov wrote:
> > ---
> >  apt/cmdline/rpmindexcopy.cc | 4 ++--
> >  1 file changed, 2 insertions(+), 2 deletions(-)
> > 
> > diff --git a/apt/cmdline/rpmindexcopy.cc b/apt/cmdline/rpmindexcopy.cc
> > index 6d5a414..06b391a 100644
> > --- a/apt/cmdline/rpmindexcopy.cc
> > +++ b/apt/cmdline/rpmindexcopy.cc
> > @@ -180,10 +180,10 @@ bool RPMIndexCopy::CopyPackages(const string &CDROM,const string &Name,vector<st
> >  		     release.c_str());
> >  	    string TargetF = _config->FindDir("Dir::State::lists") + "partial/";
> >  	    TargetF += URItoFileName(S);
> > -	    if (FileExists(RipDirectory(*I) + release) == true)
> > +	    if (FileExists(RipDirectory(*I) + "/" + release) == true)
> >  	    {
> >  	       FileFd Target(TargetF,FileFd::WriteEmpty);
> > -	       FileFd Rel(RipDirectory(*I) + release,FileFd::ReadOnly);
> > +	       FileFd Rel(RipDirectory(*I) + "/" + release,FileFd::ReadOnly);
> >  	       if (_error->PendingError() == true)
> >  		  return false;
> 
> LGTM.
> 
> I wonder is there any more bugs in constructing path names in the code.

I'd also add something like "Fixes: https://bugzilla.altlinux.org/37531".


-- 
ldv