ACK: [PATCH 2/3] uefibootpath: add test for the wireless device path

[Colin Ian King]

On 20/11/15 08:54, Ivan Hu wrote:
> Sync up with uefi spec. 2.5 for new messaging device path, wireless device path
> 
> Signed-off-by: Ivan Hu <ivan.hu at canonical.com>
> ---
>  src/uefi/uefibootpath/uefibootpath.c | 16 ++++++++++++++++
>  1 file changed, 16 insertions(+)
> 
> diff --git a/src/uefi/uefibootpath/uefibootpath.c b/src/uefi/uefibootpath/uefibootpath.c
> index fd7293c..2fd32d8 100644
> --- a/src/uefi/uefibootpath/uefibootpath.c
> +++ b/src/uefi/uefibootpath/uefibootpath.c
> @@ -604,6 +604,22 @@ static int uefibootpath_check_dev_path(fwts_framework *fw, fwts_uefi_dev_path *d
>  				errors++;
>  			}
>  			break;
> +		case FWTS_UEFI_WIRELESS_DEVICE_PATH_SUBTYPE:
> +			if (len <= sizeof(fwts_uefi_wireless_dev_path)) {
> +				fwts_failed(fw, LOG_LEVEL_MEDIUM, "UEFIWirelessDevPathLength",
> +					"The length of wireless Device Path is %" PRIu16 " bytes, "
> +					"should at least include the SSID filed.",
> +					len);
> +				errors++;
> +			}
> +			if (len > (sizeof(fwts_uefi_wireless_dev_path) + 32)) {
> +				fwts_failed(fw, LOG_LEVEL_MEDIUM, "UEFIWirelessDevPathLength",
> +					"The length of wireless Device Path is %" PRIu16 " bytes, "
> +					"the ssid field should not be larger than 32 bytes.",
> +					len);
> +				errors++;
> +			}
> +			break;
>  		default:
>  			fwts_log_info_verbatum(fw, "Unknown subtype of Messaging Device Path.");
>  			break;
> 
Acked-by: Colin Ian King <colin.king at canonical.com>