April 9, 2026
7:53 a.m.
On Thu Apr 9 12:53:40 2026 +0000, Vishnunithyasoundhar S wrote:
Changed length of tmplongpath to maximum that can be reached by extended file length path I'm a bit concerned about allocating 32KB on the stack, as that's quite large. Since adding proper length validation would make this patch more complex, I understand why you avoided it here.
Would you leave the buffer size unchanged in this patch and create a follow-up patch that allocates the 32KB temporary buffer on the heap instead? That way we can avoid the stack usage issue without expanding the scope of this merge request. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/10192#note_135663