n lib/libalpm/dload.c: https://gitlab.archlinux.org/pacman/pacman/-/merge_requests/348#note_421531

 >  					}
 >  				}
 > -

another unneeded whitespace change.

--
  
Allan McRae started a new discussion on lib/libalpm/dload.c: https://gitlab.archlinux.org/pacman/pacman/-/merge_requests/348#note_421532

 >  #else
 > -		RET_ERR(handle, ALPM_ERR_EXTERNAL_DOWNLOAD, -1);
 > +		GOTO_ERR(handle, ALPM_ERR_EXTERNAL_DOWNLOAD, cleanup);

Add `ret=-1` on the line above.

--
  
Allan McRae started a new discussion on lib/libalpm/dload.c: https://gitlab.archlinux.org/pacman/pacman/-/merge_requests/348#note_421533

 >  
 > -					MALLOC(sig_fileurl, sig_len, RET_ERR(handle, ALPM_ERR_MEMORY, -1));
 > +					MALLOC(sig_fileurl, sig_len, GOTO_ERR(handle, ALPM_ERR_MEMORY, cleanup));

We need `ret` set to `-1` here.


-- 
Reply to this email directly or view it on GitLab: https://gitlab.archlinux.org/pacman/pacman/-/merge_requests/348
You're receiving this email because of your account on gitlab.archlinux.org.Pacman | libalpm: removes stray temporary download directories (solves #297) (!348)"Allan McRae (@allan)" <gitlab@gitlab.archlinux.org> undefinedhi@josie.lol undefinedŠ.†›j