kdump-steamos: Strengthen /etc/default/kdump checks
Instead of just checking the file existence, check if it's not zero-sized. Signed-off-by: Guilherme G. Piccoli <gpiccoli@igalia.com>
This commit is contained in:
@ -46,7 +46,7 @@ grub_update() {
|
|||||||
fi
|
fi
|
||||||
}
|
}
|
||||||
|
|
||||||
if [ ! -f "/etc/default/kdump" ]; then
|
if [ ! -s "/etc/default/kdump" ]; then
|
||||||
logger "kdump-steamos: /etc/default/kdump not present - aborting..."
|
logger "kdump-steamos: /etc/default/kdump not present - aborting..."
|
||||||
exit 0
|
exit 0
|
||||||
fi
|
fi
|
||||||
|
|||||||
@ -19,7 +19,7 @@ installkernel() {
|
|||||||
|
|
||||||
install() {
|
install() {
|
||||||
# Having a valid /etc/default/kdump is essential for kdump.
|
# Having a valid /etc/default/kdump is essential for kdump.
|
||||||
if [ ! -f "/etc/default/kdump" ]; then
|
if [ ! -s "/etc/default/kdump" ]; then
|
||||||
return 1
|
return 1
|
||||||
fi
|
fi
|
||||||
|
|
||||||
|
|||||||
@ -24,7 +24,7 @@ save_locally_and_bail() {
|
|||||||
|
|
||||||
# We do some validation to be sure KDUMP_MNT pointed path is valid...
|
# We do some validation to be sure KDUMP_MNT pointed path is valid...
|
||||||
# That and having a valid /etc/default/kdump are essential conditions.
|
# That and having a valid /etc/default/kdump are essential conditions.
|
||||||
if [ ! -f "/etc/default/kdump" ]; then
|
if [ ! -s "/etc/default/kdump" ]; then
|
||||||
logger "kdump-steamos: /etc/default/kdump not present - aborting..."
|
logger "kdump-steamos: /etc/default/kdump not present - aborting..."
|
||||||
exit 0
|
exit 0
|
||||||
fi
|
fi
|
||||||
|
|||||||
Reference in New Issue
Block a user