manuel
/
live-build
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Fix cache poisoning issue in lh_binary_debian-installer with 404 downloads 

When wget 404's, an target file is created - this poisons the cache as the
file is apparently downloaded but in actuality is simply empty. We fix this
by unconditionally deleting the cache file if wget fails and emitting an
error.
This commit is contained in:
Chris Lamb 2008-07-26 00:27:29 +01:00 committed by Daniel Baumann
parent 1b37572080
commit b9c6f0a083
1 changed files with 7 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

8
helpers/lh_binary_debian-installer
View File

@ -207,7 +207,13 @@ Download_file () {
if [ ! -f "${_LH_CACHE_FILE}" ]
then
mkdir -p ${_LH_CACHE_DIR}
wget ${WGET_OPTIONS} -O "${_LH_CACHE_FILE}" "${_LH_URL}"
if ! wget ${WGET_OPTIONS} -O "${_LH_CACHE_FILE}" "${_LH_URL}"
then
rm -f "${_LH_CACHE_FILE}"
Echo_error "Could not download file: ${_LH_URL}"
exit 1
fi
fi
cp -fl "${_LH_CACHE_FILE}" "${_LH_TARGET}"