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@xxxxxxxxxx>
Looks good to me,
Reviewed-by: Darrick J. Wong <djwong@xxxxxxxxxx>
--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
>
