Re: [PATCH 1/2] mm/huge_memory: skip invalid debugfs file entry for folio split

[Date Prev] [Date Next] [Thread Prev] [Thread Next] [Date Index] [Thread Index]

Subject: Re: [PATCH 1/2] mm/huge_memory: skip invalid debugfs file entry for folio split
From: Andrew Morton <akpm@xxxxxxxxxxxxxxxxxxxx>
Date: Thu, 25 Apr 2024 14:01:26 -0700
Cc: ziy@xxxxxxxxxx, linux-mm@xxxxxxxxx, fstests@xxxxxxxxxxxxxxx, linux-xfs@xxxxxxxxxxxxxxx, linux-kernel@xxxxxxxxxxxxxxx, willy@xxxxxxxxxxxxx, hare@xxxxxxx, john.g.garry@xxxxxxxxxx, p.raghav@xxxxxxxxxxx, da.gomez@xxxxxxxxxxx
In-reply-to: <20240424225449.1498244-2-mcgrof@kernel.org>
References: <20240424225449.1498244-1-mcgrof@kernel.org> <20240424225449.1498244-2-mcgrof@kernel.org>

On Wed, 24 Apr 2024 15:54:48 -0700 Luis Chamberlain <mcgrof@xxxxxxxxxx> wrote:

> If the file entry is too long we may easily end up going out of bounds
> and crash after strsep() on sscanf(). 
> 

Can you explain why?  I'm not seeing it.

Follow-Ups:
- Re: [PATCH 1/2] mm/huge_memory: skip invalid debugfs file entry for folio split
  - From: Luis Chamberlain

References:
- [PATCH 0/2] mm/huge_memory: couple fixes and one cleanup
  - From: Luis Chamberlain
- [PATCH 1/2] mm/huge_memory: skip invalid debugfs file entry for folio split
  - From: Luis Chamberlain

Prev by Date: Re: [PATCH] mm/huge_memory: move writeback and truncation checks early
Next by Date: Re: [PATCH 1/2] mm/huge_memory: skip invalid debugfs file entry for folio split
Previous by thread: Re: [PATCH 1/2] mm/huge_memory: skip invalid debugfs file entry for folio split
Next by thread: Re: [PATCH 1/2] mm/huge_memory: skip invalid debugfs file entry for folio split
Index(es):
- Date
- Thread

[Index of Archives] [Linux Filesystems Development] [Linux NFS] [Linux NILFS] [Linux USB Devel] [Linux Audio Users] [Yosemite News] [Linux Kernel] [Linux SCSI]

Powered by Linux