[PATCH] uefidump: add Relative Offset Range subtype support for Media device path (LP: #1322465)

[IvanHu]

On 05/23/2014 11:47 PM, Colin Ian King wrote:
> On 23/05/14 09:39, Ivan Hu wrote:
>> Add the subtype support.
>> Sub-Type 8 – Relative Offset Range.
>>
>> Signed-off-by: Ivan Hu <ivan.hu at canonical.com>
>> ---
>>   src/uefi/uefidump/uefidump.c |    8 ++++++++
>>   1 file changed, 8 insertions(+)
>>
>> diff --git a/src/uefi/uefidump/uefidump.c b/src/uefi/uefidump/uefidump.c
>> index 61e8b7c..efb2c53 100644
>> --- a/src/uefi/uefidump/uefidump.c
>> +++ b/src/uefi/uefidump/uefidump.c
>> @@ -573,6 +573,14 @@ static char *uefidump_build_dev_path(char *path, fwts_uefi_dev_path *dev_path, c
>>   					p->fw_volume_name.info4[6], p->fw_volume_name.info4[7]);
>>   			}
>>   			break;
>> +		case FWTS_UEFI_RELATIVE_OFFSET_RANGE_SUBTYPE:
>> +			if (dev_path_len >= sizeof(fwts_relative_offset_range_path)) {
>> +				fwts_relative_offset_range_path *r = (fwts_relative_offset_range_path *)dev_path;
>> +				path = uefidump_vprintf(path, "\\RELATIVEOFFSETRANGE("
>> +					"0x%" PRIx64 ",0x%" PRIx64 ")",
>> +					r->starting_offset, r->ending_offset);
>> +			}
>> +			break;
>>   		default:
>>   			path = uefidump_vprintf(path, "\\Unknown-MEDIA-DEV-PATH(0x%" PRIx8 ")", dev_path->subtype);
>>   			break;
>>
>
> For some reason I can't get this to compile. I think my repo is not in
> sync with what you are building on. Perhaps we need some patches ack'd
> and applied to the fwts git report and I'll try again later next week.
>
> Colin
>

The fwts_relative_offset_range_path is defined in the fwts_uefi.h and 
included in the patch, [V2] uefi: add a test for sanity checking for the 
UEFI Boot path (LP: #1316019).
you might need to apply it first.

Ivan