linux-stable.git/block/partitions, branch linux-3.14.y

mac: validate mac_partition is within sector

2016-03-03T23:06:49+00:00

commit 02e2a5bfebe99edcf9d694575a75032d53fe1b73 upstream.

If md->signature == MAC_DRIVER_MAGIC and md->block_size == 1023, a single
512 byte sector would be read (secsize / 512). However the partition
structure would be located past the end of the buffer (secsize % 512).

Signed-off-by: Kees Cook 
Signed-off-by: Jens Axboe 
Signed-off-by: Greg Kroah-Hartman

partitions: aix.c: off by one bug

2014-10-05T21:52:24+00:00

commit d97a86c170b4e432f76db072a827fe30b4d6f659 upstream.

The lvip[] array has "state->limit" elements so the condition here
should be >= instead of >.

Fixes: 6ceea22bbbc8 ('partitions: add aix lvm partition support files')
Signed-off-by: Dan Carpenter 
Acked-by: Philippe De Muyter 
Signed-off-by: Jens Axboe 
Signed-off-by: Greg Kroah-Hartman

block/partitions/efi.c: fix bound check

2013-11-22T00:42:27+00:00

Use ARRAY_SIZE instead of sizeof to get proper max for label length.

Since this is just a read out of bounds it's not that bad, but the
problem becomes user-visible eg if one tries to use DEBUG_PAGEALLOC and
DEBUG_RODATA, at least with some enhancements from Hiroshi.  Of course
the destination array can contain garbage when we read beyond the end of
source array so that would be another user-visible problem.

Signed-off-by: Antti P Miettinen 
Reviewed-by: Hiroshi Doyu 
Tested-by: Hiroshi Doyu 
Cc: Will Drewry 
Cc: Matt Fleming 
Acked-by: Davidlohr Bueso 
Signed-off-by: Andrew Morton 
Signed-off-by: Linus Torvalds

block/partitions/efi.c: treat size mismatch as a warning, not an error

2013-10-17T04:35:53+00:00

In commit 27a7c642174e ("partitions/efi: account for pmbr size in lba")
we started treating bad sizes in lba field of the partition that has the
0xEE (GPT protective) as errors.

However, we may run into these "bad sizes" in the real world if someone
uses dd to copy an image from a smaller disk to a bigger disk.  Since
this case used to work (even without using force_gpt), keep it working
and treat the size mismatch as a warning instead of an error.

Reported-by: Josh Triplett 
Reported-by: Sean Paul 
Signed-off-by: Doug Anderson 
Reviewed-by: Josh Triplett 
Acked-by: Davidlohr Bueso 
Tested-by: Artem Bityutskiy 
Signed-off-by: Andrew Morton 
Signed-off-by: Linus Torvalds

block: change config option name for cmdline partition parsing

2013-09-30T21:31:02+00:00

Recently commit bab55417b10c ("block: support embedded device command
line partition") introduced CONFIG_CMDLINE_PARSER.  However, that name
is too generic and sounds like it enables/disables generic kernel boot
arg processing, when it really is block specific.

Before this option becomes a part of a full/final release, add the BLK_
prefix to it so that it is clear in absence of any other context that it
is block specific.

In addition, fix up the following less critical items:
 - help text was not really at all helpful.
 - index file for Documentation was not updated
 - add the new arg to Documentation/kernel-parameters.txt
 - clarify wording in source comments

Signed-off-by: Paul Gortmaker 
Cc: Jens Axboe 
Cc: Cai Zhiyong 
Cc: Wei Yongjun 
Signed-off-by: Andrew Morton 
Signed-off-by: Linus Torvalds

partitions/efi: loosen check fot pmbr size in lba

2013-09-15T11:10:16+00:00

Matt found that commit 27a7c642174e ("partitions/efi: account for pmbr
size in lba") caused his GPT formatted eMMC device not to boot.  The
reason is that this commit enforced Linux to always check the lesser of
the whole disk or 2Tib for the pMBR size in LBA.  While most disk
partitioning tools out there create a pMBR with these characteristics,
Microsoft does not, as it always sets the entry to the maximum 32-bit
limitation - even though a drive may be smaller than that[1].

Loosen this check and only verify that the size is either the whole disk
or 0xFFFFFFFF.  No tool in its right mind would set it to any value
other than these.

[1] http://thestarman.pcministry.com/asm/mbr/GPT.htm#GPTPT

Reported-and-tested-by: Matt Porter 
Signed-off-by: Davidlohr Bueso 
Signed-off-by: Linus Torvalds

block/partitions/efi.c: consistently use pr_foo()

2013-09-11T22:59:19+00:00

Cc: Davidlohr Bueso 
Cc: Karel Zak 
Cc: Matt Fleming 
Signed-off-by: Andrew Morton 
Signed-off-by: Linus Torvalds

partitions/efi: some style cleanups

2013-09-11T22:59:19+00:00

Trivial coding style cleanups - still plenty left.

[akpm@linux-foundation.org: coding-style fixes]
Signed-off-by: Davidlohr Bueso 
Reviewed-by: Karel Zak 
Acked-by: Matt Fleming 
Signed-off-by: Andrew Morton 
Signed-off-by: Linus Torvalds

partitions/efi: delete annoying emacs style comments

2013-09-11T22:59:18+00:00

I love emacs, but these settings for coding style are annoying when trying
to open the efi.h file.  More important, we already have checkpatch for
that.

Signed-off-by: Davidlohr Bueso 
Reviewed-by: Karel Zak 
Acked-by: Matt Fleming 
Signed-off-by: Andrew Morton 
Signed-off-by: Linus Torvalds

partitions/efi: compare first and last usable LBAs

2013-09-11T22:59:18+00:00

When verifying GPT header integrity, make sure that first usable LBA is
smaller than last usable LBA.

Signed-off-by: Davidlohr Bueso 
Reviewed-by: Karel Zak 
Acked-by: Matt Fleming 
Signed-off-by: Andrew Morton 
Signed-off-by: Linus Torvalds