The ext2fs_test_block_bitmap_range2 func is not a void, so make sure we
return a value. I picked EINVAL as it looks like what other things in
this func use.
URL: http://bugs.gentoo.org/467986
Reported-by: Vicente Olivert Riera <[email protected]>
Signed-off-by: Mike Frysinger <[email protected]>
---
lib/ext2fs/gen_bitmap64.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/lib/ext2fs/gen_bitmap64.c b/lib/ext2fs/gen_bitmap64.c
index 42a97d4..44ac499 100644
--- a/lib/ext2fs/gen_bitmap64.c
+++ b/lib/ext2fs/gen_bitmap64.c
@@ -658,7 +658,7 @@ int ext2fs_test_block_bitmap_range2(ext2fs_block_bitmap bmap,
if ((block < bmap->start) || (block+num-1 > bmap->end)) {
ext2fs_warn_bitmap(EXT2_ET_BAD_BLOCK_TEST, block,
bmap->description);
- return;
+ return EINVAL;
}
return bmap->bitmap_ops->test_clear_bmap_extent(bmap, block, num);
--
1.8.2.1
On Sat, May 04, 2013 at 01:05:54AM -0400, Mike Frysinger wrote:
> The ext2fs_test_block_bitmap_range2 func is not a void, so make sure we
> return a value. I picked EINVAL as it looks like what other things in
> this func use.
>
> URL: http://bugs.gentoo.org/467986
> Reported-by: Vicente Olivert Riera <[email protected]>
> Signed-off-by: Mike Frysinger <[email protected]>
This is already fixed in e2fsprogs's git repository, in commit
0d89e5acea9.
Cheers,
- Ted
On Saturday 04 May 2013 19:55:03 Theodore Ts'o wrote:
> On Sat, May 04, 2013 at 01:05:54AM -0400, Mike Frysinger wrote:
> > The ext2fs_test_block_bitmap_range2 func is not a void, so make sure we
> > return a value. I picked EINVAL as it looks like what other things in
> > this func use.
> >
> > URL: http://bugs.gentoo.org/467986
> > Reported-by: Vicente Olivert Riera <[email protected]>
> > Signed-off-by: Mike Frysinger <[email protected]>
>
> This is already fixed in e2fsprogs's git repository, in commit
> 0d89e5acea9.
ok, the use of "master" and "next" is weird. i was using the normal "master"
branch which is apparently out of date.
-mike
On 2013-05-04, at 18:58, Mike Frysinger <[email protected]> wrote:
> On Saturday 04 May 2013 19:55:03 Theodore Ts'o wrote:
>> On Sat, May 04, 2013 at 01:05:54AM -0400, Mike Frysinger wrote:
>>> The ext2fs_test_block_bitmap_range2 func is not a void, so make sure we
>>> return a value. I picked EINVAL as it looks like what other things in
>>> this func use.
>>>
>>> URL: http://bugs.gentoo.org/467986
>>> Reported-by: Vicente Olivert Riera <[email protected]>
>>> Signed-off-by: Mike Frysinger <[email protected]>
>>
>> This is already fixed in e2fsprogs's git repository, in commit
>> 0d89e5acea9.
>
> ok, the use of "master" and "next" is weird. i was using the normal "master"
> branch which is apparently out of date.
I'd have to agree. It would be great if master was merged from maint more often so that patches would apply to either.
Cheers, Andreas
On Sat, May 04, 2013 at 08:40:21PM -0600, Andreas Dilger wrote:
> > ok, the use of "master" and "next" is weird. i was using the normal "master"
> > branch which is apparently out of date.
>
> I'd have to agree. It would be great if master was merged from maint more often so that patches would apply to either.
Sorry, I merge maint into next, but sometimes I forget to bump master
up.
Fixed now.
- Ted