Staging: hv: file2alias: fix up alias creation logic for hv_vmbus_device_id
Greg Kroah-Hartman [Thu, 25 Aug 2011 18:28:11 +0000 (11:28 -0700)]
When I added the driver_data field to hv_vmbus_device_id, I forgot to
take into the account how the alias was created, so it would append the
kernel pointer to the end of the alias, which is not correct.

This changes how the hv_vmbus_device_id alias is created to proper
account for the driver_data field.  As no module yet uses this alias, it
is safe to fix this up at this point in the commit stream.

Cc: K. Y. Srinivasan <kys@microsoft.com>
Cc: Haiyang Zhang <haiyangz@microsoft.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>

scripts/mod/file2alias.c

index b74d21a..f936d1f 100644 (file)
@@ -745,9 +745,9 @@ static int do_vmbus_entry(const char *filename, struct hv_vmbus_device_id *id,
                          char *alias)
 {
        int i;
-       char guid_name[((sizeof(struct hv_vmbus_device_id) + 1)) * 2];
+       char guid_name[((sizeof(id->guid) + 1)) * 2];
 
-       for (i = 0; i < (sizeof(struct hv_vmbus_device_id) * 2); i += 2)
+       for (i = 0; i < (sizeof(id->guid) * 2); i += 2)
                sprintf(&guid_name[i], "%02x", id->guid[i/2]);
 
        strcpy(alias, "vmbus:");