Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]

Groups > linux.kernel > #1651296 > unrolled thread

[PATCH v3 1/1] partitions/msdos: FreeBSD UFS2 file systems are not recognized

Started byRichard Narron <comet.berkeley@gmail.com>
First post2017-05-26 12:50 +0200
Last post2017-05-27 03:40 +0200
Articles 8 — 3 participants

Back to article view | Back to linux.kernel

Contents

  [PATCH v3 1/1] partitions/msdos: FreeBSD UFS2 file systems are not  recognized Richard Narron <comet.berkeley@gmail.com> - 2017-05-26 12:50 +0200
    Re: [PATCH v3 1/1] partitions/msdos: FreeBSD UFS2 file systems are  not recognized Jens Axboe <axboe@kernel.dk> - 2017-05-26 15:30 +0200
    Re: [PATCH v3 1/1] partitions/msdos: FreeBSD UFS2 file systems are  not recognized Joe Perches <joe@perches.com> - 2017-05-26 16:40 +0200
      Re: [PATCH v3 1/1] partitions/msdos: FreeBSD UFS2 file systems are  not recognized Joe Perches <joe@perches.com> - 2017-05-27 03:30 +0200
        Re: [PATCH v3 1/1] partitions/msdos: FreeBSD UFS2 file systems are  not recognized Joe Perches <joe@perches.com> - 2017-05-27 04:00 +0200
          Re: [PATCH v3 1/1] partitions/msdos: FreeBSD UFS2 file systems are  not recognized Richard Narron <comet.berkeley@gmail.com> - 2017-05-27 05:30 +0200
            Re: [PATCH v3 1/1] partitions/msdos: FreeBSD UFS2 file systems are  not recognized Joe Perches <joe@perches.com> - 2017-05-27 06:30 +0200
      Re: [PATCH v3 1/1] partitions/msdos: FreeBSD UFS2 file systems are  not recognized Richard Narron <comet.berkeley@gmail.com> - 2017-05-27 03:40 +0200

#1651296 — [PATCH v3 1/1] partitions/msdos: FreeBSD UFS2 file systems are not recognized

FromRichard Narron <comet.berkeley@gmail.com>
Date2017-05-26 12:50 +0200
Subject[PATCH v3 1/1] partitions/msdos: FreeBSD UFS2 file systems are not recognized
Message-ID<tLkYF-4LU-1@gated-at.bofh.it>
The code in block/partitions/msdos.c recognizes FreeBSD, OpenBSD
and NetBSD partitions and does a reasonable job picking out OpenBSD
and NetBSD UFS subpartitions.

But for FreeBSD the subpartitions are always "bad".

     Kernel: <bsd:bad subpartition - ignored

Though all 3 of these BSD systems use UFS as a file system, only
FreeBSD uses relative start addresses in the subpartition
declarations.

The following patch fixes this for FreeBSD partitions and leaves
the code for OpenBSD and NetBSD intact:

Cc: Jens Axboe <axboe@kernel.dk>
Cc: Andries Brouwer <aeb@cwi.nl>
Cc: linux-block@vger.kernel.org
Reviewed-by: Christoph Hellwig <hch@lst.de>
Signed-off-by: Richard Narron <comet.berkeley@gmail.com>
---
Changelog v2->v3:
- Add Cc:
Changelog v1->v2: - Improve style, use +=
---
  block/partitions/msdos.c | 2 ++
  1 file changed, 2 insertions(+)

--- a/block/partitions/msdos.c	2015-12-27 18:17:37.000000000 -0800
+++ b/block/partitions/msdos.c	2015-12-29 10:44:25.813773357 -0800
@@ -300,6 +300,8 @@ static void parse_bsd(struct parsed_part
  			continue;
  		bsd_start = le32_to_cpu(p->p_offset);
  		bsd_size = le32_to_cpu(p->p_size);
+		if (memcmp(flavour, "bsd\0", 4) == 0)
+			bsd_start += offset;
  		if (offset == bsd_start && size == bsd_size)
  			/* full parent partition, we have it already */
  			continue;

[toc] | [next] | [standalone]

#1651391 — Re: [PATCH v3 1/1] partitions/msdos: FreeBSD UFS2 file systems are not recognized

FromJens Axboe <axboe@kernel.dk>
Date2017-05-26 15:30 +0200
SubjectRe: [PATCH v3 1/1] partitions/msdos: FreeBSD UFS2 file systems are not recognized
Message-ID<tLntw-6pE-9@gated-at.bofh.it>
In reply to#1651296
On 05/26/2017 04:48 AM, Richard Narron wrote:
> The code in block/partitions/msdos.c recognizes FreeBSD, OpenBSD
> and NetBSD partitions and does a reasonable job picking out OpenBSD
> and NetBSD UFS subpartitions.
> 
> But for FreeBSD the subpartitions are always "bad".
> 
>      Kernel: <bsd:bad subpartition - ignored
> 
> Though all 3 of these BSD systems use UFS as a file system, only
> FreeBSD uses relative start addresses in the subpartition
> declarations.
> 
> The following patch fixes this for FreeBSD partitions and leaves
> the code for OpenBSD and NetBSD intact:

I queued this up 3 days ago, and replied as such. There's no need
to keep sending it.

-- 
Jens Axboe

[toc] | [prev] | [next] | [standalone]

#1651414 — Re: [PATCH v3 1/1] partitions/msdos: FreeBSD UFS2 file systems are not recognized

FromJoe Perches <joe@perches.com>
Date2017-05-26 16:40 +0200
SubjectRe: [PATCH v3 1/1] partitions/msdos: FreeBSD UFS2 file systems are not recognized
Message-ID<tLozf-73D-11@gated-at.bofh.it>
In reply to#1651296
On Fri, 2017-05-26 at 03:48 -0700, Richard Narron wrote:
> The code in block/partitions/msdos.c recognizes FreeBSD, OpenBSD
> and NetBSD partitions and does a reasonable job picking out OpenBSD
> and NetBSD UFS subpartitions.
> 
> But for FreeBSD the subpartitions are always "bad".
> 
>      Kernel: <bsd:bad subpartition - ignored
[]
>   block/partitions/msdos.c | 2 ++
[]
> @@ -300,6 +300,8 @@ static void parse_bsd(struct parsed_part
>   			continue;
>   		bsd_start = le32_to_cpu(p->p_offset);
>   		bsd_size = le32_to_cpu(p->p_size);
> +		if (memcmp(flavour, "bsd\0", 4) == 0)

Weird code.  Why not:

		if (strcmp(flavor, "bsd") == 0)

[toc] | [prev] | [next] | [standalone]

#1651674 — Re: [PATCH v3 1/1] partitions/msdos: FreeBSD UFS2 file systems are not recognized

FromJoe Perches <joe@perches.com>
Date2017-05-27 03:30 +0200
SubjectRe: [PATCH v3 1/1] partitions/msdos: FreeBSD UFS2 file systems are not recognized
Message-ID<tLyIm-4Vw-147@gated-at.bofh.it>
In reply to#1651414
On Fri, 2017-05-26 at 16:30 -0700, Richard Narron wrote:
> On Fri, 26 May 2017, Joe Perches wrote:
> 
> > On Fri, 2017-05-26 at 03:48 -0700, Richard Narron wrote:
> > > The code in block/partitions/msdos.c recognizes FreeBSD, OpenBSD
> > > and NetBSD partitions and does a reasonable job picking out OpenBSD
> > > and NetBSD UFS subpartitions.
> > > 
> > > But for FreeBSD the subpartitions are always "bad".
> > > 
> > >      Kernel: <bsd:bad subpartition - ignored
> > 
> > []
> > >   block/partitions/msdos.c | 2 ++
> > 
> > []
> > > @@ -300,6 +300,8 @@ static void parse_bsd(struct parsed_part
> > >   			continue;
> > >   		bsd_start = le32_to_cpu(p->p_offset);
> > >   		bsd_size = le32_to_cpu(p->p_size);
> > > +		if (memcmp(flavour, "bsd\0", 4) == 0)
> > 
> > Weird code.  Why not:
> > 
> > 		if (strcmp(flavor, "bsd") == 0)
> > 
> 
> I instinctively trust the memcmp function as it seems more like 
> assembly language to me and more straight forward and more reliable than 
> strcmp.

That really doesn't matter.

Your code stores "bsd\0\0" and not just "bsd\0"

[toc] | [prev] | [next] | [standalone]

#1651723 — Re: [PATCH v3 1/1] partitions/msdos: FreeBSD UFS2 file systems are not recognized

FromJoe Perches <joe@perches.com>
Date2017-05-27 04:00 +0200
SubjectRe: [PATCH v3 1/1] partitions/msdos: FreeBSD UFS2 file systems are not recognized
Message-ID<tLzbj-55K-3@gated-at.bofh.it>
In reply to#1651674
(please keep replies on the list)

On Fri, 2017-05-26 at 18:33 -0700, Richard Narron wrote:
> On Fri, 26 May 2017, Joe Perches wrote:
> > On Fri, 2017-05-26 at 16:30 -0700, Richard Narron wrote:
> > > On Fri, 26 May 2017, Joe Perches wrote:
> > > > On Fri, 2017-05-26 at 03:48 -0700, Richard Narron wrote:
> > > > > The code in block/partitions/msdos.c recognizes FreeBSD, OpenBSD
> > > > > and NetBSD partitions and does a reasonable job picking out OpenBSD
> > > > > and NetBSD UFS subpartitions.
> > > > > 
> > > > > But for FreeBSD the subpartitions are always "bad".
> > > > > 
> > > > >      Kernel: <bsd:bad subpartition - ignored
> > > > 
> > > > []
> > > > >   block/partitions/msdos.c | 2 ++
> > > > 
> > > > []
> > > > > @@ -300,6 +300,8 @@ static void parse_bsd(struct parsed_part
> > > > >   			continue;
> > > > >   		bsd_start = le32_to_cpu(p->p_offset);
> > > > >   		bsd_size = le32_to_cpu(p->p_size);
> > > > > +		if (memcmp(flavour, "bsd\0", 4) == 0)
> > > > 
> > > > Weird code.  Why not:
> > > > 
> > > > 		if (strcmp(flavor, "bsd") == 0)
> > > > 
> > > 
> > > I instinctively trust the memcmp function as it seems more like
> > > assembly language to me and more straight forward and more reliable than
> > > strcmp.
> > 
> > That really doesn't matter.
> > 
> > Your code stores "bsd\0\0" and not just "bsd\0"
> > 
> 
> Thanks for looking at this code. I do appreciate it.
> 
> How about saving a byte and doing this instead?
> 
>            if (memcmp(flavour, "bsd", 4) == 0)
> 
> I do appreciate your input as coding style is important, but so too is 
> reliability.
> 
> I don't trust the string functions and probably never will.
> 
> It is not surprising to me that things like SQL injection and any number of other 
> C string exploits are very common.
> 
> IBM gave up on the idea of marking memory to keep track of data length with the 1401 machines in the 1950's.
> 
> But Digital Equipment kept the idea alive of using null characters for a 
> long time.  Sadly the C programming language copied this bad idea for 
> strings.

Let's not argue the language.

Please use what's normal for the language as that is
readers of the code typically expect.

[toc] | [prev] | [next] | [standalone]

#1651741 — Re: [PATCH v3 1/1] partitions/msdos: FreeBSD UFS2 file systems are not recognized

FromRichard Narron <comet.berkeley@gmail.com>
Date2017-05-27 05:30 +0200
SubjectRe: [PATCH v3 1/1] partitions/msdos: FreeBSD UFS2 file systems are not recognized
Message-ID<tLAAp-6f2-5@gated-at.bofh.it>
In reply to#1651723
On Fri, 26 May 2017, Joe Perches wrote:

> (please keep replies on the list)
>
> On Fri, 2017-05-26 at 18:33 -0700, Richard Narron wrote:
>> On Fri, 26 May 2017, Joe Perches wrote:
>>> On Fri, 2017-05-26 at 16:30 -0700, Richard Narron wrote:
>>>> On Fri, 26 May 2017, Joe Perches wrote:
>>>>> On Fri, 2017-05-26 at 03:48 -0700, Richard Narron wrote:
>>>>>> The code in block/partitions/msdos.c recognizes FreeBSD, OpenBSD
>>>>>> and NetBSD partitions and does a reasonable job picking out OpenBSD
>>>>>> and NetBSD UFS subpartitions.
>>>>>>
>>>>>> But for FreeBSD the subpartitions are always "bad".
>>>>>>
>>>>>>      Kernel: <bsd:bad subpartition - ignored
>>>>>
>>>>> []
>>>>>>   block/partitions/msdos.c | 2 ++
>>>>>
>>>>> []
>>>>>> @@ -300,6 +300,8 @@ static void parse_bsd(struct parsed_part
>>>>>>   			continue;
>>>>>>   		bsd_start = le32_to_cpu(p->p_offset);
>>>>>>   		bsd_size = le32_to_cpu(p->p_size);
>>>>>> +		if (memcmp(flavour, "bsd\0", 4) == 0)
>>>>>
>>>>> Weird code.  Why not:
>>>>>
>>>>> 		if (strcmp(flavor, "bsd") == 0)
>>>>>
>>>>
>>>> I instinctively trust the memcmp function as it seems more like
>>>> assembly language to me and more straight forward and more reliable than
>>>> strcmp.
>>>
>>> That really doesn't matter.
>>>
>>> Your code stores "bsd\0\0" and not just "bsd\0"
>>>
>>
>> Thanks for looking at this code. I do appreciate it.
>>
>> How about saving a byte and doing this instead?
>>
>>            if (memcmp(flavour, "bsd", 4) == 0)
>>
>> I do appreciate your input as coding style is important, but so too is
>> reliability.
>>
>> I don't trust the string functions and probably never will.
>>
>> It is not surprising to me that things like SQL injection and any number of other
>> C string exploits are very common.
>>
>> IBM gave up on the idea of marking memory to keep track of data length with the 1401 machines in the 1950's.
>>
>> But Digital Equipment kept the idea alive of using null characters for a
>> long time.  Sadly the C programming language copied this bad idea for
>> strings.
>
> Let's not argue the language.
>
> Please use what's normal for the language as that is
> readers of the code typically expect.
>

Under the /block/partitions directory the c programs have about 13 uses 
of memcmp() and 6 uses of strcmp().

[toc] | [prev] | [next] | [standalone]

#1651751 — Re: [PATCH v3 1/1] partitions/msdos: FreeBSD UFS2 file systems are not recognized

FromJoe Perches <joe@perches.com>
Date2017-05-27 06:30 +0200
SubjectRe: [PATCH v3 1/1] partitions/msdos: FreeBSD UFS2 file systems are not recognized
Message-ID<tLBwu-6RP-1@gated-at.bofh.it>
In reply to#1651741
On Fri, 2017-05-26 at 20:20 -0700, Richard Narron wrote:
> Under the /block/partitions directory the c programs have about 13 uses 
> of memcmp() and 6 uses of strcmp().

Nearly all of the memcmp uses with strings kernel wide use
the equivalent of memcmp(foo, "bar", strlen("bar"));

[toc] | [prev] | [next] | [standalone]

#1651694 — Re: [PATCH v3 1/1] partitions/msdos: FreeBSD UFS2 file systems are not recognized

FromRichard Narron <comet.berkeley@gmail.com>
Date2017-05-27 03:40 +0200
SubjectRe: [PATCH v3 1/1] partitions/msdos: FreeBSD UFS2 file systems are not recognized
Message-ID<tLyIm-4Vw-149@gated-at.bofh.it>
In reply to#1651414
On Fri, 26 May 2017, Joe Perches wrote:

> On Fri, 2017-05-26 at 03:48 -0700, Richard Narron wrote:
>> The code in block/partitions/msdos.c recognizes FreeBSD, OpenBSD
>> and NetBSD partitions and does a reasonable job picking out OpenBSD
>> and NetBSD UFS subpartitions.
>>
>> But for FreeBSD the subpartitions are always "bad".
>>
>>      Kernel: <bsd:bad subpartition - ignored
> []
>>   block/partitions/msdos.c | 2 ++
> []
>> @@ -300,6 +300,8 @@ static void parse_bsd(struct parsed_part
>>   			continue;
>>   		bsd_start = le32_to_cpu(p->p_offset);
>>   		bsd_size = le32_to_cpu(p->p_size);
>> +		if (memcmp(flavour, "bsd\0", 4) == 0)
>
> Weird code.  Why not:
>
> 		if (strcmp(flavor, "bsd") == 0)
>

I instinctively trust the memcmp function as it seems more like 
assembly language to me and more straight forward and more reliable than 
strcmp.

I'm new to this forum and did not know who you were so I looked up your 
name in old threads and oddly enough found this thread about a strcmp bug:

http://marc.info/?l=linux-kernel&m=125848558316468&w=2

http://marc.info/?l=linux-kernel&m=125847802903422&w=2

[toc] | [prev] | [standalone]

Back to top | Article view | linux.kernel

csiph-web