* Re: [PATCH v2] libxcmd: add return value check for dynamic memory function
@ 2023-06-08 5:40 Christoph Hellwig
0 siblings, 0 replies; 6+ messages in thread
From: Christoph Hellwig @ 2023-06-08 5:40 UTC (permalink / raw)
To: Weifeng Su; +Cc: linux-xfs, djwong, hch, sandeen, linfeilong, liuzhiqiang26
Still looks good:
Reviewed-by: Christoph Hellwig <hch@lst.de>
^ permalink raw reply [flat|nested] 6+ messages in thread
* [PATCH v2] libxcmd: add return value check for dynamic memory function
@ 2023-06-08 2:51 Weifeng Su
2023-06-08 14:46 ` Darrick J. Wong
0 siblings, 1 reply; 6+ messages in thread
From: Weifeng Su @ 2023-06-08 2:51 UTC (permalink / raw)
To: linux-xfs, djwong, hch, sandeen; +Cc: linfeilong, liuzhiqiang26, Weifeng Su
The result check was missed and It cause the coredump like:
0x00005589f3e358dd in add_command (ci=0x5589f3e3f020 <health_cmd>) at command.c:37
0x00005589f3e337d8 in init_commands () at init.c:37
init (argc=<optimized out>, argv=0x7ffecfb0cd28) at init.c:102
0x00005589f3e33399 in main (argc=<optimized out>, argv=<optimized out>) at init.c:112
Add check for realloc function to ignore this coredump and exit with
error output
Signed-off-by: Weifeng Su <suweifeng1@huawei.com>
---
Changes since version 1:
- Modify according to review opinions, Add more string
libxcmd/command.c | 4 ++++
1 file changed, 4 insertions(+)
diff --git a/libxcmd/command.c b/libxcmd/command.c
index a76d1515..e2603097 100644
--- a/libxcmd/command.c
+++ b/libxcmd/command.c
@@ -34,6 +34,10 @@ add_command(
const cmdinfo_t *ci)
{
cmdtab = realloc((void *)cmdtab, ++ncmds * sizeof(*cmdtab));
+ if (!cmdtab) {
+ perror(_("adding libxcmd command"));
+ exit(1);
+ }
cmdtab[ncmds - 1] = *ci;
qsort(cmdtab, ncmds, sizeof(*cmdtab), compare);
}
--
2.18.0.windows.1
^ permalink raw reply related [flat|nested] 6+ messages in thread* Re: [PATCH v2] libxcmd: add return value check for dynamic memory function
2023-06-08 2:51 Weifeng Su
@ 2023-06-08 14:46 ` Darrick J. Wong
2023-06-12 12:23 ` Weifeng Su
0 siblings, 1 reply; 6+ messages in thread
From: Darrick J. Wong @ 2023-06-08 14:46 UTC (permalink / raw)
To: Weifeng Su; +Cc: linux-xfs, hch, sandeen, linfeilong, liuzhiqiang26
On Thu, Jun 08, 2023 at 10:51:46AM +0800, Weifeng Su wrote:
> The result check was missed and It cause the coredump like:
> 0x00005589f3e358dd in add_command (ci=0x5589f3e3f020 <health_cmd>) at command.c:37
> 0x00005589f3e337d8 in init_commands () at init.c:37
> init (argc=<optimized out>, argv=0x7ffecfb0cd28) at init.c:102
> 0x00005589f3e33399 in main (argc=<optimized out>, argv=<optimized out>) at init.c:112
>
> Add check for realloc function to ignore this coredump and exit with
> error output
>
> Signed-off-by: Weifeng Su <suweifeng1@huawei.com>
Looks good to me,
Reviewed-by: Darrick J. Wong <djwong@kernel.org>
--D
> ---
> Changes since version 1:
> - Modify according to review opinions, Add more string
>
> libxcmd/command.c | 4 ++++
> 1 file changed, 4 insertions(+)
>
> diff --git a/libxcmd/command.c b/libxcmd/command.c
> index a76d1515..e2603097 100644
> --- a/libxcmd/command.c
> +++ b/libxcmd/command.c
> @@ -34,6 +34,10 @@ add_command(
> const cmdinfo_t *ci)
> {
> cmdtab = realloc((void *)cmdtab, ++ncmds * sizeof(*cmdtab));
> + if (!cmdtab) {
> + perror(_("adding libxcmd command"));
> + exit(1);
> + }
> cmdtab[ncmds - 1] = *ci;
> qsort(cmdtab, ncmds, sizeof(*cmdtab), compare);
> }
> --
> 2.18.0.windows.1
>
^ permalink raw reply [flat|nested] 6+ messages in thread* Re: [PATCH v2] libxcmd: add return value check for dynamic memory function
2023-06-08 14:46 ` Darrick J. Wong
@ 2023-06-12 12:23 ` Weifeng Su
2023-06-12 15:25 ` Darrick J. Wong
0 siblings, 1 reply; 6+ messages in thread
From: Weifeng Su @ 2023-06-12 12:23 UTC (permalink / raw)
To: Darrick J. Wong; +Cc: linux-xfs, hch, sandeen, linfeilong, liuzhiqiang26
Just ping, Is the merge windown open?
On 2023/6/8 22:46, Darrick J. Wong wrote:
> On Thu, Jun 08, 2023 at 10:51:46AM +0800, Weifeng Su wrote:
>> The result check was missed and It cause the coredump like:
>> 0x00005589f3e358dd in add_command (ci=0x5589f3e3f020 <health_cmd>) at command.c:37
>> 0x00005589f3e337d8 in init_commands () at init.c:37
>> init (argc=<optimized out>, argv=0x7ffecfb0cd28) at init.c:102
>> 0x00005589f3e33399 in main (argc=<optimized out>, argv=<optimized out>) at init.c:112
>>
>> Add check for realloc function to ignore this coredump and exit with
>> error output
>>
>> Signed-off-by: Weifeng Su <suweifeng1@huawei.com>
>
> Looks good to me,
> Reviewed-by: Darrick J. Wong <djwong@kernel.org>
>
> --D
>
>> ---
>> Changes since version 1:
>> - Modify according to review opinions, Add more string
>>
>> libxcmd/command.c | 4 ++++
>> 1 file changed, 4 insertions(+)
>>
>> diff --git a/libxcmd/command.c b/libxcmd/command.c
>> index a76d1515..e2603097 100644
>> --- a/libxcmd/command.c
>> +++ b/libxcmd/command.c
>> @@ -34,6 +34,10 @@ add_command(
>> const cmdinfo_t *ci)
>> {
>> cmdtab = realloc((void *)cmdtab, ++ncmds * sizeof(*cmdtab));
>> + if (!cmdtab) {
>> + perror(_("adding libxcmd command"));
>> + exit(1);
>> + }
>> cmdtab[ncmds - 1] = *ci;
>> qsort(cmdtab, ncmds, sizeof(*cmdtab), compare);
>> }
>> --
>> 2.18.0.windows.1
>>
^ permalink raw reply [flat|nested] 6+ messages in thread* Re: [PATCH v2] libxcmd: add return value check for dynamic memory function
2023-06-12 12:23 ` Weifeng Su
@ 2023-06-12 15:25 ` Darrick J. Wong
2023-06-13 7:52 ` Carlos Maiolino
0 siblings, 1 reply; 6+ messages in thread
From: Darrick J. Wong @ 2023-06-12 15:25 UTC (permalink / raw)
To: Weifeng Su, Carlos Maiolino
Cc: linux-xfs, hch, sandeen, linfeilong, liuzhiqiang26
[add xfsprogs maintainer to thread]
On Mon, Jun 12, 2023 at 08:23:00PM +0800, Weifeng Su wrote:
> Just ping, Is the merge windown open?
It's never not open for xfsprogs, but mostly it depends on the xfsprogs
maintainer (Carlos) deciding to pick up a patch.
--D
> On 2023/6/8 22:46, Darrick J. Wong wrote:
> > On Thu, Jun 08, 2023 at 10:51:46AM +0800, Weifeng Su wrote:
> > > The result check was missed and It cause the coredump like:
> > > 0x00005589f3e358dd in add_command (ci=0x5589f3e3f020 <health_cmd>) at command.c:37
> > > 0x00005589f3e337d8 in init_commands () at init.c:37
> > > init (argc=<optimized out>, argv=0x7ffecfb0cd28) at init.c:102
> > > 0x00005589f3e33399 in main (argc=<optimized out>, argv=<optimized out>) at init.c:112
> > >
> > > Add check for realloc function to ignore this coredump and exit with
> > > error output
> > >
> > > Signed-off-by: Weifeng Su <suweifeng1@huawei.com>
> >
> > Looks good to me,
> > Reviewed-by: Darrick J. Wong <djwong@kernel.org>
> >
> > --D
> >
> > > ---
> > > Changes since version 1:
> > > - Modify according to review opinions, Add more string
> > >
> > > libxcmd/command.c | 4 ++++
> > > 1 file changed, 4 insertions(+)
> > >
> > > diff --git a/libxcmd/command.c b/libxcmd/command.c
> > > index a76d1515..e2603097 100644
> > > --- a/libxcmd/command.c
> > > +++ b/libxcmd/command.c
> > > @@ -34,6 +34,10 @@ add_command(
> > > const cmdinfo_t *ci)
> > > {
> > > cmdtab = realloc((void *)cmdtab, ++ncmds * sizeof(*cmdtab));
> > > + if (!cmdtab) {
> > > + perror(_("adding libxcmd command"));
> > > + exit(1);
> > > + }
> > > cmdtab[ncmds - 1] = *ci;
> > > qsort(cmdtab, ncmds, sizeof(*cmdtab), compare);
> > > }
> > > --
> > > 2.18.0.windows.1
> > >
^ permalink raw reply [flat|nested] 6+ messages in thread* Re: [PATCH v2] libxcmd: add return value check for dynamic memory function
2023-06-12 15:25 ` Darrick J. Wong
@ 2023-06-13 7:52 ` Carlos Maiolino
0 siblings, 0 replies; 6+ messages in thread
From: Carlos Maiolino @ 2023-06-13 7:52 UTC (permalink / raw)
To: Darrick J. Wong
Cc: Weifeng Su, linux-xfs, hch, sandeen, linfeilong, liuzhiqiang26
On Mon, Jun 12, 2023 at 08:25:33AM -0700, Darrick J. Wong wrote:
> [add xfsprogs maintainer to thread]
>
> On Mon, Jun 12, 2023 at 08:23:00PM +0800, Weifeng Su wrote:
> > Just ping, Is the merge windown open?
>
> It's never not open for xfsprogs, but mostly it depends on the xfsprogs
> maintainer (Carlos) deciding to pick up a patch.
I'll pull this into for-next on my next batch. Hopefully this week yet.
--
Carlos
>
> --D
>
> > On 2023/6/8 22:46, Darrick J. Wong wrote:
> > > On Thu, Jun 08, 2023 at 10:51:46AM +0800, Weifeng Su wrote:
> > > > The result check was missed and It cause the coredump like:
> > > > 0x00005589f3e358dd in add_command (ci=0x5589f3e3f020 <health_cmd>) at command.c:37
> > > > 0x00005589f3e337d8 in init_commands () at init.c:37
> > > > init (argc=<optimized out>, argv=0x7ffecfb0cd28) at init.c:102
> > > > 0x00005589f3e33399 in main (argc=<optimized out>, argv=<optimized out>) at init.c:112
> > > >
> > > > Add check for realloc function to ignore this coredump and exit with
> > > > error output
> > > >
> > > > Signed-off-by: Weifeng Su <suweifeng1@huawei.com>
> > >
> > > Looks good to me,
> > > Reviewed-by: Darrick J. Wong <djwong@kernel.org>
> > >
> > > --D
> > >
> > > > ---
> > > > Changes since version 1:
> > > > - Modify according to review opinions, Add more string
> > > >
> > > > libxcmd/command.c | 4 ++++
> > > > 1 file changed, 4 insertions(+)
> > > >
> > > > diff --git a/libxcmd/command.c b/libxcmd/command.c
> > > > index a76d1515..e2603097 100644
> > > > --- a/libxcmd/command.c
> > > > +++ b/libxcmd/command.c
> > > > @@ -34,6 +34,10 @@ add_command(
> > > > const cmdinfo_t *ci)
> > > > {
> > > > cmdtab = realloc((void *)cmdtab, ++ncmds * sizeof(*cmdtab));
> > > > + if (!cmdtab) {
> > > > + perror(_("adding libxcmd command"));
> > > > + exit(1);
> > > > + }
> > > > cmdtab[ncmds - 1] = *ci;
> > > > qsort(cmdtab, ncmds, sizeof(*cmdtab), compare);
> > > > }
> > > > --
> > > > 2.18.0.windows.1
> > > >
^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2023-06-13 7:52 UTC | newest]
Thread overview: 6+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2023-06-08 5:40 [PATCH v2] libxcmd: add return value check for dynamic memory function Christoph Hellwig
-- strict thread matches above, loose matches on Subject: below --
2023-06-08 2:51 Weifeng Su
2023-06-08 14:46 ` Darrick J. Wong
2023-06-12 12:23 ` Weifeng Su
2023-06-12 15:25 ` Darrick J. Wong
2023-06-13 7:52 ` Carlos Maiolino
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox