proxmox-mirrors/fwupd
1
0
Fork 0
mirror of https://git.proxmox.com/git/fwupd synced 2025-08-07 12:25:38 +00:00
Code Issues Actions Packages Projects Releases Wiki Activity

trivial: Return a more invalid value if FWUPD_IS_DEVICE() fails

Browse Source 
We don't want to return something that's actually the initial value.
This commit is contained in:
Richard Hughes 2022-07-21 19:59:54 +01:00
parent ac3f7f5521
commit 74e9b04b39
2 changed files with 2 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
libfwupd/fwupd-device.c
View File

@ -1284,7 +1284,7 @@ guint32
fwupd_device_get_battery_level(FwupdDevice *self) fwupd_device_get_battery_level(FwupdDevice *self)
{ {
FwupdDevicePrivate *priv = GET_PRIVATE(self); FwupdDevicePrivate *priv = GET_PRIVATE(self);
g_return_val_if_fail(FWUPD_IS_DEVICE(self), FWUPD_BATTERY_LEVEL_INVALID); g_return_val_if_fail(FWUPD_IS_DEVICE(self), G_MAXUINT);
return priv->battery_level; return priv->battery_level;
} }

2
libfwupdplugin/fu-device.c
View File

@ -3558,7 +3558,7 @@ fu_device_ensure_battery_inhibit(FuDevice *self)
guint guint
fu_device_get_battery_level(FuDevice *self) fu_device_get_battery_level(FuDevice *self)
{ {
g_return_val_if_fail(FU_IS_DEVICE(self), FWUPD_BATTERY_LEVEL_INVALID); g_return_val_if_fail(FU_IS_DEVICE(self), G_MAXUINT);
/* use the parent if the child is unset */ /* use the parent if the child is unset */
if (fu_device_has_internal_flag(self, FU_DEVICE_INTERNAL_FLAG_USE_PARENT_FOR_BATTERY) && if (fu_device_has_internal_flag(self, FU_DEVICE_INTERNAL_FLAG_USE_PARENT_FOR_BATTERY) &&