commit baf4c4bdeff4d21c564ae89698538efb8ef4d8b2
author Szymon Starzycki <sstar@google.com> Fri Oct 04 10:28:03 2013 -0700
committer Colin Cross <ccross@android.com> Wed Dec 04 15:00:52 2013 -0800
tree 6d9fa217679ef1562a2d2afc83662dc59f09936f
parent 55b61f4c9bac7f53289c79f22b4ad0ac80ecfb02

Fastbootd: Comments and general cleaning

Alignment fix for fastbootd gpt_mmap

Change-Id: I86482e1645672daae1c8e4a4455c6291c0805fb0

fastbootd/commands/partitions.c

diff --git a/fastbootd/commands/partitions.c b/fastbootd/commands/partitions.c
index ca66c6d..de80ea3 100644
--- a/fastbootd/commands/partitions.c
+++ b/fastbootd/commands/partitions.c
@@ -61,7 +61,7 @@
     0x87, 0xc0, 0x68, 0xb6, 0xb7, 0x26, 0x99, 0xc7,
 };
 
-//TODO: If both blocks are invalid should I leave everything to vendor (through libvendor)
+//TODO: There is assumption that we are using little endian
 
 static void GPT_entry_clear(struct GPT_entry_raw *entry)
 {
@@ -83,6 +83,7 @@
         D(ERR, "the location of mapping area is outside of the device size %lld", sz);
         return 1;
     }
+    location = ALIGN_DOWN(location, PAGE_SIZE);
 
     mapping->map_ptr = mmap64(NULL, mapping->size, PROT_READ | PROT_WRITE, MAP_SHARED, fd, location);
 
@@ -291,7 +292,7 @@
     }
     D(DEBUG, "Deleting gpt entry '%s'\n", raw->partition_guid);
 
-    // This entry can be empty in the middle
+    // Entry in the middle of table may become empty
     GPT_entry_clear(raw);
 
     return 0;
@@ -382,7 +383,6 @@
 
 struct GPT_entry_raw *GPT_get_pointer_by_name(struct GPT_entry_table *table, const char *name)
 {
-    //TODO: reverse direction
     int count = (int) table->header->entries_count;
     int current;
 
@@ -560,10 +560,9 @@
 
 void entry_set_guid(int n, uint8_t *guid)
 {
-    guid[0] = (uint8_t) (n + 1);
     int fd;
     fd = open("/dev/urandom", O_RDONLY);
-    read(fd, &guid[1], 15);
+    read(fd, guid, 16);
     close(fd);
 
     //rfc4122