https://jira.whamcloud.com This file is an XML representation of an issue en-us 9.4.14 940014 05-12-2023 https://jira.whamcloud.com/browse/LU-16605 Lustre <p>When using '<tt>lfs fid2path</tt>' to find all hardlinks of a FID, the links are printed out with newline characters separating them. This is unsafe, since file paths are allowed to contain newline characters as well, which makes fid2path unreliable for non-interactive use (i.e. as part of other tools, without human supervision).</p> <p>This issue is present in the master branch, so I assume it might affect all current versions.</p> <p>To mitigate this problem and not break backwards compatibility, perhaps one could add a new command line argument that changes the path separator from '\n' to '\0'? Similar to "<tt>find . -print0</tt>", for example.</p> <p><tt># Suggested usage:</tt><br/> <tt>lfs fid2path -0 &lt;FID&gt;</tt></p> <p> </p> LU-16605

lfs fid2path should allow null byte separated output

Bug Minor Resolved Fixed Arshad Hussain Simon Westersund easy lug23dd Wed, 1 Mar 2023 08:15:47 +0000 Fri, 8 Sep 2023 05:51:44 +0000 Thu, 31 Aug 2023 15:05:54 +0000 Upstream Lustre 2.16.0 0 4 <p>I'll attach a patch suggestion based on our current 2.12.8_ddn18 (DDN Exascaler 5.2.6) version: <span class="nobr"><a href="https://jira.whamcloud.com/secure/attachment/48285/48285_lfs-fid2path.patch" title="lfs-fid2path.patch attached to LU-16605">lfs-fid2path.patch^{<img class="rendericon" src="https://jira.whamcloud.com/images/icons/link_attachment_7.gif" height="7" width="7" align="absmiddle" alt="" border="0"/>}</a></span></p> <p>It should be fairly easy to adapt this to the master branch.</p> <p>I think it is prett reasonable to have a "<tt>-0</tt>" option for this. </p> <p>One related improvement that I've wanted for a while is to allow printing only the filename(s) of the file instead of the whole pathname. That can be useful for debugging if the pathname is broken for some reason. There is already "<tt>-f</tt>" to print the FID, so something like "<tt>-n</tt>" to print the name(s) only. Both "<tt>-f</tt>" and "<tt>-n</tt>" could be used together. </p> <p>"Arshad Hussain &lt;arshad.hussain@aeoncomputing.com&gt;" uploaded a new patch: <a href="https://review.whamcloud.com/c/fs/lustre-release/+/51626" class="external-link" target="_blank" rel="nofollow noopener">https://review.whamcloud.com/c/fs/lustre-release/+/51626</a><br/> Subject: <a href="https://jira.whamcloud.com/browse/LU-16605" title="lfs fid2path should allow null byte separated output" class="issue-link" data-issue-key="LU-16605"><del>LU-16605</del></a> lfs: Add -n option to fid2path<br/> Project: fs/lustre-release<br/> Branch: master<br/> Current Patch Set: 1<br/> Commit: 16272732f122b903d7f2896182fb626aad216d40</p> <p>"Arshad Hussain &lt;arshad.hussain@aeoncomputing.com&gt;" uploaded a new patch: <a href="https://review.whamcloud.com/c/fs/lustre-release/+/51736" class="external-link" target="_blank" rel="nofollow noopener">https://review.whamcloud.com/c/fs/lustre-release/+/51736</a><br/> Subject: <a href="https://jira.whamcloud.com/browse/LU-16605" title="lfs fid2path should allow null byte separated output" class="issue-link" data-issue-key="LU-16605"><del>LU-16605</del></a> lfs: Add -0 option to fid2path<br/> Project: fs/lustre-release<br/> Branch: master<br/> Current Patch Set: 1<br/> Commit: dd442430705836fece9a51b5833cad8e21ce10ac</p> <p>"Oleg Drokin &lt;green@whamcloud.com&gt;" merged in patch <a href="https://review.whamcloud.com/c/fs/lustre-release/+/51736/" class="external-link" target="_blank" rel="nofollow noopener">https://review.whamcloud.com/c/fs/lustre-release/+/51736/</a><br/> Subject: <a href="https://jira.whamcloud.com/browse/LU-16605" title="lfs fid2path should allow null byte separated output" class="issue-link" data-issue-key="LU-16605"><del>LU-16605</del></a> lfs: Add -0 option to fid2path<br/> Project: fs/lustre-release<br/> Branch: master<br/> Current Patch Set: <br/> Commit: 8d4f9d1befb9962335d4cbc5b89cafced286b066</p> <p>"Oleg Drokin &lt;green@whamcloud.com&gt;" merged in patch <a href="https://review.whamcloud.com/c/fs/lustre-release/+/51626/" class="external-link" target="_blank" rel="nofollow noopener">https://review.whamcloud.com/c/fs/lustre-release/+/51626/</a><br/> Subject: <a href="https://jira.whamcloud.com/browse/LU-16605" title="lfs fid2path should allow null byte separated output" class="issue-link" data-issue-key="LU-16605"><del>LU-16605</del></a> lfs: Add -n option to fid2path<br/> Project: fs/lustre-release<br/> Branch: master<br/> Current Patch Set: <br/> Commit: a1bc7a84ce5b3b649fd590a80645f299205024c2</p> <p>Landed for 2.16</p> Related Development Rank 1|i03fd3: Rank (Obsolete) 9223372036854775807 Severity