ACPI: fix ACPI=n allmodconfig build
[safe/jmp/linux-2.6] / fs / partitions / ldm.c
index 99873a2..8652fb9 100644 (file)
 #ifndef CONFIG_LDM_DEBUG
 #define ldm_debug(...) do {} while (0)
 #else
-#define ldm_debug(f, a...) _ldm_printk (KERN_DEBUG, __FUNCTION__, f, ##a)
+#define ldm_debug(f, a...) _ldm_printk (KERN_DEBUG, __func__, f, ##a)
 #endif
 
-#define ldm_crit(f, a...)  _ldm_printk (KERN_CRIT,  __FUNCTION__, f, ##a)
-#define ldm_error(f, a...) _ldm_printk (KERN_ERR,   __FUNCTION__, f, ##a)
-#define ldm_info(f, a...)  _ldm_printk (KERN_INFO,  __FUNCTION__, f, ##a)
+#define ldm_crit(f, a...)  _ldm_printk (KERN_CRIT,  __func__, f, ##a)
+#define ldm_error(f, a...) _ldm_printk (KERN_ERR,   __func__, f, ##a)
+#define ldm_info(f, a...)  _ldm_printk (KERN_INFO,  __func__, f, ##a)
 
 __attribute__ ((format (printf, 3, 4)))
 static void _ldm_printk (const char *level, const char *function,
@@ -133,17 +133,17 @@ static bool ldm_parse_privhead(const u8 *data, struct privhead *ph)
        bool is_vista = false;
 
        BUG_ON(!data || !ph);
-       if (MAGIC_PRIVHEAD != BE64(data)) {
+       if (MAGIC_PRIVHEAD != get_unaligned_be64(data)) {
                ldm_error("Cannot find PRIVHEAD structure. LDM database is"
                        " corrupt. Aborting.");
                return false;
        }
-       ph->ver_major = BE16(data + 0x000C);
-       ph->ver_minor = BE16(data + 0x000E);
-       ph->logical_disk_start = BE64(data + 0x011B);
-       ph->logical_disk_size = BE64(data + 0x0123);
-       ph->config_start = BE64(data + 0x012B);
-       ph->config_size = BE64(data + 0x0133);
+       ph->ver_major = get_unaligned_be16(data + 0x000C);
+       ph->ver_minor = get_unaligned_be16(data + 0x000E);
+       ph->logical_disk_start = get_unaligned_be64(data + 0x011B);
+       ph->logical_disk_size = get_unaligned_be64(data + 0x0123);
+       ph->config_start = get_unaligned_be64(data + 0x012B);
+       ph->config_size = get_unaligned_be64(data + 0x0133);
        /* Version 2.11 is Win2k/XP and version 2.12 is Vista. */
        if (ph->ver_major == 2 && ph->ver_minor == 12)
                is_vista = true;
@@ -191,14 +191,14 @@ static bool ldm_parse_tocblock (const u8 *data, struct tocblock *toc)
 {
        BUG_ON (!data || !toc);
 
-       if (MAGIC_TOCBLOCK != BE64 (data)) {
+       if (MAGIC_TOCBLOCK != get_unaligned_be64(data)) {
                ldm_crit ("Cannot find TOCBLOCK, database may be corrupt.");
                return false;
        }
        strncpy (toc->bitmap1_name, data + 0x24, sizeof (toc->bitmap1_name));
        toc->bitmap1_name[sizeof (toc->bitmap1_name) - 1] = 0;
-       toc->bitmap1_start = BE64 (data + 0x2E);
-       toc->bitmap1_size  = BE64 (data + 0x36);
+       toc->bitmap1_start = get_unaligned_be64(data + 0x2E);
+       toc->bitmap1_size  = get_unaligned_be64(data + 0x36);
 
        if (strncmp (toc->bitmap1_name, TOC_BITMAP1,
                        sizeof (toc->bitmap1_name)) != 0) {
@@ -208,8 +208,8 @@ static bool ldm_parse_tocblock (const u8 *data, struct tocblock *toc)
        }
        strncpy (toc->bitmap2_name, data + 0x46, sizeof (toc->bitmap2_name));
        toc->bitmap2_name[sizeof (toc->bitmap2_name) - 1] = 0;
-       toc->bitmap2_start = BE64 (data + 0x50);
-       toc->bitmap2_size  = BE64 (data + 0x58);
+       toc->bitmap2_start = get_unaligned_be64(data + 0x50);
+       toc->bitmap2_size  = get_unaligned_be64(data + 0x58);
        if (strncmp (toc->bitmap2_name, TOC_BITMAP2,
                        sizeof (toc->bitmap2_name)) != 0) {
                ldm_crit ("TOCBLOCK's second bitmap is '%s', should be '%s'.",
@@ -237,22 +237,22 @@ static bool ldm_parse_vmdb (const u8 *data, struct vmdb *vm)
 {
        BUG_ON (!data || !vm);
 
-       if (MAGIC_VMDB != BE32 (data)) {
+       if (MAGIC_VMDB != get_unaligned_be32(data)) {
                ldm_crit ("Cannot find the VMDB, database may be corrupt.");
                return false;
        }
 
-       vm->ver_major = BE16 (data + 0x12);
-       vm->ver_minor = BE16 (data + 0x14);
+       vm->ver_major = get_unaligned_be16(data + 0x12);
+       vm->ver_minor = get_unaligned_be16(data + 0x14);
        if ((vm->ver_major != 4) || (vm->ver_minor != 10)) {
                ldm_error ("Expected VMDB version %d.%d, got %d.%d. "
                        "Aborting.", 4, 10, vm->ver_major, vm->ver_minor);
                return false;
        }
 
-       vm->vblk_size     = BE32 (data + 0x08);
-       vm->vblk_offset   = BE32 (data + 0x0C);
-       vm->last_vblk_seq = BE32 (data + 0x04);
+       vm->vblk_size     = get_unaligned_be32(data + 0x08);
+       vm->vblk_offset   = get_unaligned_be32(data + 0x0C);
+       vm->last_vblk_seq = get_unaligned_be32(data + 0x04);
 
        ldm_debug ("Parsed VMDB successfully.");
        return true;
@@ -507,7 +507,7 @@ static bool ldm_validate_vmdb (struct block_device *bdev, unsigned long base,
                goto out;                               /* Already logged */
 
        /* Are there uncommitted transactions? */
-       if (BE16(data + 0x10) != 0x01) {
+       if (get_unaligned_be16(data + 0x10) != 0x01) {
                ldm_crit ("Database is not in a consistent state.  Aborting.");
                goto out;
        }
@@ -677,15 +677,24 @@ static bool ldm_create_data_partitions (struct parsed_partitions *pp,
  * Return:  -1 Error, the calculated offset exceeded the size of the buffer
  *           n OK, a range-checked offset into buffer
  */
-static int ldm_relative (const u8 *buffer, int buflen, int base, int offset)
+static int ldm_relative(const u8 *buffer, int buflen, int base, int offset)
 {
 
        base += offset;
-       if ((!buffer) || (offset < 0) || (base > buflen))
+       if (!buffer || offset < 0 || base > buflen) {
+               if (!buffer)
+                       ldm_error("!buffer");
+               if (offset < 0)
+                       ldm_error("offset (%d) < 0", offset);
+               if (base > buflen)
+                       ldm_error("base (%d) > buflen (%d)", base, buflen);
                return -1;
-       if ((base + buffer[base]) >= buflen)
+       }
+       if (base + buffer[base] >= buflen) {
+               ldm_error("base (%d) + buffer[base] (%d) >= buflen (%d)", base,
+                               buffer[base], buflen);
                return -1;
-
+       }
        return buffer[base] + offset + 1;
 }
 
@@ -793,7 +802,7 @@ static bool ldm_parse_cmp3 (const u8 *buffer, int buflen, struct vblk *vb)
                return false;
 
        len += VBLK_SIZE_CMP3;
-       if (len != BE32 (buffer + 0x14))
+       if (len != get_unaligned_be32(buffer + 0x14))
                return false;
 
        comp = &vb->vblk.comp;
@@ -842,7 +851,7 @@ static int ldm_parse_dgr3 (const u8 *buffer, int buflen, struct vblk *vb)
                return false;
 
        len += VBLK_SIZE_DGR3;
-       if (len != BE32 (buffer + 0x14))
+       if (len != get_unaligned_be32(buffer + 0x14))
                return false;
 
        dgrp = &vb->vblk.dgrp;
@@ -886,7 +895,7 @@ static bool ldm_parse_dgr4 (const u8 *buffer, int buflen, struct vblk *vb)
                return false;
 
        len += VBLK_SIZE_DGR4;
-       if (len != BE32 (buffer + 0x14))
+       if (len != get_unaligned_be32(buffer + 0x14))
                return false;
 
        dgrp = &vb->vblk.dgrp;
@@ -922,7 +931,7 @@ static bool ldm_parse_dsk3 (const u8 *buffer, int buflen, struct vblk *vb)
                return false;
 
        len += VBLK_SIZE_DSK3;
-       if (len != BE32 (buffer + 0x14))
+       if (len != get_unaligned_be32(buffer + 0x14))
                return false;
 
        disk = &vb->vblk.disk;
@@ -959,7 +968,7 @@ static bool ldm_parse_dsk4 (const u8 *buffer, int buflen, struct vblk *vb)
                return false;
 
        len += VBLK_SIZE_DSK4;
-       if (len != BE32 (buffer + 0x14))
+       if (len != get_unaligned_be32(buffer + 0x14))
                return false;
 
        disk = &vb->vblk.disk;
@@ -1025,14 +1034,14 @@ static bool ldm_parse_prt3(const u8 *buffer, int buflen, struct vblk *vb)
                return false;
        }
        len += VBLK_SIZE_PRT3;
-       if (len > BE32(buffer + 0x14)) {
+       if (len > get_unaligned_be32(buffer + 0x14)) {
                ldm_error("len %d > BE32(buffer + 0x14) %d", len,
-                               BE32(buffer + 0x14));
+                               get_unaligned_be32(buffer + 0x14));
                return false;
        }
        part = &vb->vblk.part;
-       part->start = BE64(buffer + 0x24 + r_name);
-       part->volume_offset = BE64(buffer + 0x2C + r_name);
+       part->start = get_unaligned_be64(buffer + 0x24 + r_name);
+       part->volume_offset = get_unaligned_be64(buffer + 0x2C + r_name);
        part->size = ldm_get_vnum(buffer + 0x34 + r_name);
        part->parent_id = ldm_get_vnum(buffer + 0x34 + r_size);
        part->disk_id = ldm_get_vnum(buffer + 0x34 + r_parent);
@@ -1054,60 +1063,98 @@ static bool ldm_parse_prt3(const u8 *buffer, int buflen, struct vblk *vb)
  * Return:  'true'   @vb contains a Volume VBLK
  *          'false'  @vb contents are not defined
  */
-static bool ldm_parse_vol5 (const u8 *buffer, int buflen, struct vblk *vb)
+static bool ldm_parse_vol5(const u8 *buffer, int buflen, struct vblk *vb)
 {
-       int r_objid, r_name, r_vtype, r_child, r_size, r_id1, r_id2, r_size2;
-       int r_drive, len;
+       int r_objid, r_name, r_vtype, r_disable_drive_letter, r_child, r_size;
+       int r_id1, r_id2, r_size2, r_drive, len;
        struct vblk_volu *volu;
 
-       BUG_ON (!buffer || !vb);
-
-       r_objid  = ldm_relative (buffer, buflen, 0x18, 0);
-       r_name   = ldm_relative (buffer, buflen, 0x18, r_objid);
-       r_vtype  = ldm_relative (buffer, buflen, 0x18, r_name);
-       r_child  = ldm_relative (buffer, buflen, 0x2E, r_vtype);
-       r_size   = ldm_relative (buffer, buflen, 0x3E, r_child);
-
-       if (buffer[0x12] & VBLK_FLAG_VOLU_ID1)
-               r_id1 = ldm_relative (buffer, buflen, 0x53, r_size);
-       else
+       BUG_ON(!buffer || !vb);
+       r_objid = ldm_relative(buffer, buflen, 0x18, 0);
+       if (r_objid < 0) {
+               ldm_error("r_objid %d < 0", r_objid);
+               return false;
+       }
+       r_name = ldm_relative(buffer, buflen, 0x18, r_objid);
+       if (r_name < 0) {
+               ldm_error("r_name %d < 0", r_name);
+               return false;
+       }
+       r_vtype = ldm_relative(buffer, buflen, 0x18, r_name);
+       if (r_vtype < 0) {
+               ldm_error("r_vtype %d < 0", r_vtype);
+               return false;
+       }
+       r_disable_drive_letter = ldm_relative(buffer, buflen, 0x18, r_vtype);
+       if (r_disable_drive_letter < 0) {
+               ldm_error("r_disable_drive_letter %d < 0",
+                               r_disable_drive_letter);
+               return false;
+       }
+       r_child = ldm_relative(buffer, buflen, 0x2D, r_disable_drive_letter);
+       if (r_child < 0) {
+               ldm_error("r_child %d < 0", r_child);
+               return false;
+       }
+       r_size = ldm_relative(buffer, buflen, 0x3D, r_child);
+       if (r_size < 0) {
+               ldm_error("r_size %d < 0", r_size);
+               return false;
+       }
+       if (buffer[0x12] & VBLK_FLAG_VOLU_ID1) {
+               r_id1 = ldm_relative(buffer, buflen, 0x52, r_size);
+               if (r_id1 < 0) {
+                       ldm_error("r_id1 %d < 0", r_id1);
+                       return false;
+               }
+       } else
                r_id1 = r_size;
-
-       if (buffer[0x12] & VBLK_FLAG_VOLU_ID2)
-               r_id2 = ldm_relative (buffer, buflen, 0x53, r_id1);
-       else
+       if (buffer[0x12] & VBLK_FLAG_VOLU_ID2) {
+               r_id2 = ldm_relative(buffer, buflen, 0x52, r_id1);
+               if (r_id2 < 0) {
+                       ldm_error("r_id2 %d < 0", r_id2);
+                       return false;
+               }
+       } else
                r_id2 = r_id1;
-
-       if (buffer[0x12] & VBLK_FLAG_VOLU_SIZE)
-               r_size2 = ldm_relative (buffer, buflen, 0x53, r_id2);
-       else
+       if (buffer[0x12] & VBLK_FLAG_VOLU_SIZE) {
+               r_size2 = ldm_relative(buffer, buflen, 0x52, r_id2);
+               if (r_size2 < 0) {
+                       ldm_error("r_size2 %d < 0", r_size2);
+                       return false;
+               }
+       } else
                r_size2 = r_id2;
-
-       if (buffer[0x12] & VBLK_FLAG_VOLU_DRIVE)
-               r_drive = ldm_relative (buffer, buflen, 0x53, r_size2);
-       else
+       if (buffer[0x12] & VBLK_FLAG_VOLU_DRIVE) {
+               r_drive = ldm_relative(buffer, buflen, 0x52, r_size2);
+               if (r_drive < 0) {
+                       ldm_error("r_drive %d < 0", r_drive);
+                       return false;
+               }
+       } else
                r_drive = r_size2;
-
        len = r_drive;
-       if (len < 0)
+       if (len < 0) {
+               ldm_error("len %d < 0", len);
                return false;
-
+       }
        len += VBLK_SIZE_VOL5;
-       if (len != BE32 (buffer + 0x14))
+       if (len > get_unaligned_be32(buffer + 0x14)) {
+               ldm_error("len %d > BE32(buffer + 0x14) %d", len,
+                               get_unaligned_be32(buffer + 0x14));
                return false;
-
+       }
        volu = &vb->vblk.volu;
-
-       ldm_get_vstr (buffer + 0x18 + r_name,  volu->volume_type,
-               sizeof (volu->volume_type));
-       memcpy (volu->volume_state, buffer + 0x19 + r_vtype,
-                       sizeof (volu->volume_state));
-       volu->size = ldm_get_vnum (buffer + 0x3E + r_child);
-       volu->partition_type = buffer[0x42 + r_size];
-       memcpy (volu->guid, buffer + 0x43 + r_size,  sizeof (volu->guid));
+       ldm_get_vstr(buffer + 0x18 + r_name, volu->volume_type,
+                       sizeof(volu->volume_type));
+       memcpy(volu->volume_state, buffer + 0x18 + r_disable_drive_letter,
+                       sizeof(volu->volume_state));
+       volu->size = ldm_get_vnum(buffer + 0x3D + r_child);
+       volu->partition_type = buffer[0x41 + r_size];
+       memcpy(volu->guid, buffer + 0x42 + r_size, sizeof(volu->guid));
        if (buffer[0x12] & VBLK_FLAG_VOLU_DRIVE) {
-               ldm_get_vstr (buffer + 0x53 + r_size,  volu->drive_hint,
-                       sizeof (volu->drive_hint));
+               ldm_get_vstr(buffer + 0x52 + r_size, volu->drive_hint,
+                               sizeof(volu->drive_hint));
        }
        return true;
 }
@@ -1247,9 +1294,9 @@ static bool ldm_frag_add (const u8 *data, int size, struct list_head *frags)
 
        BUG_ON (!data || !frags);
 
-       group = BE32 (data + 0x08);
-       rec   = BE16 (data + 0x0C);
-       num   = BE16 (data + 0x0E);
+       group = get_unaligned_be32(data + 0x08);
+       rec   = get_unaligned_be16(data + 0x0C);
+       num   = get_unaligned_be16(data + 0x0E);
        if ((num < 1) || (num > 4)) {
                ldm_error ("A VBLK claims to have %d parts.", num);
                return false;
@@ -1378,12 +1425,12 @@ static bool ldm_get_vblks (struct block_device *bdev, unsigned long base,
                }
 
                for (v = 0; v < perbuf; v++, data+=size) {  /* For each vblk */
-                       if (MAGIC_VBLK != BE32 (data)) {
+                       if (MAGIC_VBLK != get_unaligned_be32(data)) {
                                ldm_error ("Expected to find a VBLK.");
                                goto out;
                        }
 
-                       recs = BE16 (data + 0x0E);      /* Number of records */
+                       recs = get_unaligned_be16(data + 0x0E); /* Number of records */
                        if (recs == 1) {
                                if (!ldm_ldmdb_add (data, size, ldb))
                                        goto out;       /* Already logged */