2014-02-19 07:14:02

by Yuan Pengfei

[permalink] [raw]
Subject: [PATCH] gcov: fix memory allocation problem in gcov_info_dup

From: Yuan Pengfei <[email protected]>

If -fprofile-values option is used, ctr->num and sci_ptr->num
may be zero, resulting in zero size and cv_size, which will
cause ENOMEM when opening gcov data files in debugfs.
Signed-off-by: Yuan Pengfei <[email protected]>
---
kernel/gcov/gcc_3_4.c | 2 +-
kernel/gcov/gcc_4_7.c | 2 +-
2 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/kernel/gcov/gcc_3_4.c b/kernel/gcov/gcc_3_4.c
index 27bc88a..1c1f425 100644
--- a/kernel/gcov/gcc_3_4.c
+++ b/kernel/gcov/gcc_3_4.c
@@ -269,7 +269,7 @@ struct gcov_info *gcov_info_dup(struct gcov_info *info)
dup->counts[i].num = ctr->num;
dup->counts[i].merge = ctr->merge;
dup->counts[i].values = vmalloc(size);
- if (!dup->counts[i].values)
+ if (size && !dup->counts[i].values)
goto err_free;
memcpy(dup->counts[i].values, ctr->values, size);
}
diff --git a/kernel/gcov/gcc_4_7.c b/kernel/gcov/gcc_4_7.c
index 2c6e463..7465944 100644
--- a/kernel/gcov/gcc_4_7.c
+++ b/kernel/gcov/gcc_4_7.c
@@ -290,7 +290,7 @@ struct gcov_info *gcov_info_dup(struct gcov_info *info)

dci_ptr->values = vmalloc(cv_size);

- if (!dci_ptr->values)
+ if (cv_size && !dci_ptr->values)
goto err_free;

dci_ptr->num = sci_ptr->num;
--
1.8.5.3????{.n?+???????+%?????ݶ??w??{.n?+????{??G?????{ay?ʇڙ?,j??f???h?????????z_??(?階?ݢj"???m??????G????????????&???~???iO???z??v?^?m???? ????????I?


2014-02-19 08:52:47

by Peter Oberparleiter

[permalink] [raw]
Subject: Re: [PATCH] gcov: fix memory allocation problem in gcov_info_dup

On 19.02.2014 08:06, Yuan Pengfei wrote:
> From: Yuan Pengfei <[email protected]>
>
> If -fprofile-values option is used, ctr->num and sci_ptr->num
> may be zero, resulting in zero size and cv_size, which will
> cause ENOMEM when opening gcov data files in debugfs.
> Signed-off-by: Yuan Pengfei <[email protected]>

I'm unaware of any user of -fprofile-values in the kernel. Are you
trying to extend gcov-kernel to also support -fprofile-values? I would
expect that additional changes to the .gcda file creation logic are
required to fully support this GCC option.


Regards,
Peter Oberparleiter

--
Peter Oberparleiter
Linux on System z Development - IBM Germany

2014-02-19 10:44:32

by Yuan Pengfei

[permalink] [raw]
Subject: Re: [PATCH] gcov: fix memory allocation problem in gcov_info_dup

Using -fprofile-values is one case that will cause the memory allocation
problem. More changes are needed to fully support -fprofile-values. I
submit this patch first because I think it is obvious and does no harm.
> I'm unaware of any user of -fprofile-values in the kernel. Are you
> trying to extend gcov-kernel to also support -fprofile-values? I would
> expect that additional changes to the .gcda file creation logic are
> required to fully support this GCC option.????{.n?+???????+%?????ݶ??w??{.n?+????{??G?????{ay?ʇڙ?,j??f???h?????????z_??(?階?ݢj"???m??????G????????????&???~???iO???z??v?^?m???? ????????I?

2014-02-20 08:52:22

by Peter Oberparleiter

[permalink] [raw]
Subject: Re: [PATCH] gcov: fix memory allocation problem in gcov_info_dup

On 19.02.2014 11:38, Yuan Pengfei wrote:
> Using -fprofile-values is one case that will cause the memory allocation
> problem. More changes are needed to fully support -fprofile-values. I
> submit this patch first because I think it is obvious and does no harm.
>> I'm unaware of any user of -fprofile-values in the kernel. Are you
>> trying to extend gcov-kernel to also support -fprofile-values? I would
>> expect that additional changes to the .gcda file creation logic are
>> required to fully support this GCC option.

If there is a way to trigger the problem of gcov_info.values == NULL
using the existing gcov-kernel infrastructure then I agree that this
should be fixed. If not, then I would rather see the suggested change as
part of a patch series that enables other profiling types.

--
Peter Oberparleiter
Linux on System z Development - IBM Germany