Bug 256201

Summary: sysutils/fusefs-ntfs: delete obsolete patch to lowntfs-3g.c
Product: Ports & Packages Reporter: Alan Somers <asomers>
Component: Individual Port(s)Assignee: Alan Somers <asomers>
Status: New ---    
Severity: Affects Many People CC: fernape, freebsd
Priority: --- Flags: bugzilla: maintainer-feedback? (freebsd)
Version: Latest   
Hardware: Any   
OS: Any   
Description Flags
Delete an obsolete patch. none

Description Alan Somers freebsd_committer 2021-05-27 15:20:51 UTC
Created attachment 225320 [details]
Delete an obsolete patch.

sysutils/fusefs-ntfs patches src/lowntfs-3g.c to remove the return action from ntfs_fuse_bmap.  But that function cannot possibly work correctly in the patched form.  It _must_ call either fuse_reply_err or fuse_reply_bmap, or else the fusefs driver will leak commands and user threads will hang.  This patch was added in 2010, so I'm guessing that ports' copy of libfuse in 2010 lacked bmap support.  That would explain the necessity of the patch.  However, it serves no purpose now, and can be deleted.

I have build tested this in Poudriere for amd64 13.0, 12.2, and 11.4.  But I haven't done any runtime testing.
Comment 1 Fernando ApesteguĂ­a freebsd_committer 2021-06-03 05:59:52 UTC
^Triage: Reporter is committer, assign accordingly.