[PATCH 0/6] solve memory leak and check whether NULL pointer

[PATCH 0/6] solve memory leak and check whether NULL pointer

[zhanchengbin]

Solve the memory leak of the abnormal branch and the new null pointer check

zhanchengbin (6):
   alloc_string : String structure consistency
   ss_execute_command : Check whether the pointer is not null before it
     is referenced.
   quota_set_sb_inum : Check whether the pointer is not null  before it
     is referenced.
   badblock_list memory leak
   ldesc Non-empty judgment
   io->name memory leak

  e2fsck/logfile.c      |  2 ++
  e2fsck/problem.c      |  2 ++
  lib/ext2fs/test_io.c  |  2 ++
  lib/ext2fs/undo_io.c  |  2 ++
  lib/ss/execute_cmd.c  |  2 ++
  lib/support/mkquota.c |  3 ++-
  misc/dumpe2fs.c       |  1 +
  resize/resize2fs.c    | 10 ++++++----
  8 files changed, 19 insertions(+), 5 deletions(-)

-- 
2.23.0

Re: [PATCH 0/6] solve memory leak and check whether NULL pointer

[Theodore Y. Ts'o]

On Thu, Dec 02, 2021 at 07:26:25PM +0800, zhanchengbin wrote:
> Solve the memory leak of the abnormal branch and the new null pointer check
> 
> zhanchengbin (6):
>   alloc_string : String structure consistency
>   ss_execute_command : Check whether the pointer is not null before it
>     is referenced.
>   quota_set_sb_inum : Check whether the pointer is not null  before it
>     is referenced.
>   badblock_list memory leak
>   ldesc Non-empty judgment
>   io->name memory leak

The cover letter for the patch series arrived, but none of the patches
associated with it did.  Could you resend?

Thanks,

					- Ted

Re: [PATCH 0/6] solve memory leak and check whether NULL pointer

[Zhiqiang Liu]

Sorry for that, we will resend the patch series.

On 2021/12/2 22:53, Theodore Y. Ts'o wrote:
> On Thu, Dec 02, 2021 at 07:26:25PM +0800, zhanchengbin wrote:
>> Solve the memory leak of the abnormal branch and the new null pointer check
>>
>> zhanchengbin (6):
>>   alloc_string : String structure consistency
>>   ss_execute_command : Check whether the pointer is not null before it
>>     is referenced.
>>   quota_set_sb_inum : Check whether the pointer is not null  before it
>>     is referenced.
>>   badblock_list memory leak
>>   ldesc Non-empty judgment
>>   io->name memory leak
> The cover letter for the patch series arrived, but none of the patches
> associated with it did.  Could you resend?
>
> Thanks,
>
> 					- Ted
>
> .

[PATCH 0/6] solve memory leak and check whether NULL pointer

[zhanchengbin]

Solve the memory leak of the abnormal branch and the new null pointer check

zhanchengbin (6):
   e2fsck: set s->len=0 if malloc() fails in alloc_string()
   lib/ss: check whether argp is null before accessing it in
     ss_execute_command()
   lib/support: check whether inump is null before accessing it in
     quota_set_sb_inum()
   e2fsprogs: call ext2fs_badblocks_list_free() to free list in exception
     branch
   e2fsck: check whether ldesc is null before accessing it in
     end_problem_latch()
   lib/ext2fs: call ext2fs_free_mem() to free &io->name in exception
     branch

  e2fsck/logfile.c      | 2 ++
  e2fsck/problem.c      | 2 ++
  lib/ext2fs/test_io.c  | 2 ++
  lib/ext2fs/undo_io.c  | 2 ++
  lib/ss/execute_cmd.c  | 2 ++
  lib/support/mkquota.c | 3 ++-
  misc/dumpe2fs.c       | 1 +
  resize/resize2fs.c    | 4 ++--
  8 files changed, 15 insertions(+), 3 deletions(-)

-- 
2.23.0