ACK: [PATCH 24/26] uefi: uefidump: cast to ptrdiff_t to remove mixed signed/unsigned comparison

Alex Hung alex.hung at canonical.com
Mon Oct 15 07:08:55 UTC 2012


On 10/15/2012 04:32 AM, Colin King wrote:
> From: Colin Ian King <colin.king at canonical.com>
>
> We shouldn't be comparing a ptrdiff_t to a size_t, so cast to avoid
> a signed/unsigned comparison.
>
> Signed-off-by: Colin Ian King <colin.king at canonical.com>
> ---
>   src/uefi/uefidump/uefidump.c |    3 ++-
>   1 file changed, 2 insertions(+), 1 deletion(-)
>
> diff --git a/src/uefi/uefidump/uefidump.c b/src/uefi/uefidump/uefidump.c
> index 11c40e9..d09db0b 100644
> --- a/src/uefi/uefidump/uefidump.c
> +++ b/src/uefi/uefidump/uefidump.c
> @@ -16,6 +16,7 @@
>    * Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301, USA.
>    *
>    */
> +#include <stddef.h>
>   #include <inttypes.h>
>
>   #include "fwts.h"
> @@ -645,7 +646,7 @@ static void uefidump_info_signature_support(fwts_framework *fw, fwts_uefi_var *v
>   		if (var->datalen)
>   			fwts_log_info_verbatum(fw, "  Signature GUIDs:");
>
> -		while (data - var->data < var->datalen) {
> +		while (data - var->data < (ptrdiff_t)var->datalen) {
>   			fwts_guid_buf_to_str(data, guid_str, sizeof(guid_str));
>   			fwts_log_info_verbatum(fw, "    %s", guid_str);
>   			data += 16;
>

Acked-by: Alex Hung <alex.hung at canonical.com>



More information about the fwts-devel mailing list