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-3427 Lustre <p>Following <a href="https://jira.whamcloud.com/browse/LU-2444" title="Fix &#39;error handling&#39; issues" class="issue-link" data-issue-key="LU-2444"><del>LU-2444</del></a>, Coverity found a few more error handling issues in the Lustre code.</p> <p>I will propose a patch to address the issues.</p> LU-3427

Fix 'error handling' issues

Technical task LU-2753 Minor Resolved Fixed Bobbie Lind Sebastien Buisson build coverity patch Fri, 31 May 2013 13:53:58 +0000 Fri, 4 Oct 2013 19:50:16 +0000 Wed, 7 Aug 2013 21:29:15 +0000 Lustre 2.4.0 Lustre 2.4.1 Lustre 2.5.0 0 5 <p>The patch is at:<br/> <a href="http://review.whamcloud.com/6510" class="external-link" target="_blank" rel="nofollow noopener">http://review.whamcloud.com/6510</a></p> <p>Thanks,<br/> Sebastien.</p> <p>Thanks Sebastien</p> <p>Bobbie</p> <p>Could you please take care of this patch?</p> <p>Thanks</p> <p>Peter</p> <p>The patch was hitting an LBUG found in <a href="https://jira.whamcloud.com/browse/LU-3453" title="LBUG in mdt_intent_opc(), (layout.c:1916:__req_capsule_get()) ASSERTION( msg != ((void *)0) ) failed" class="issue-link" data-issue-key="LU-3453"><del>LU-3453</del></a>. This bug has been fixed and I rebased and uploaded the rebased patch.</p> <p>This patch broke the compilation of master and was reverted. After speaking with Oleg, the variable that this patch references no longer exists. If you feel the patch is still needed please submit it again.</p> <p>Variable no longer in code base.</p> <p>Of course I feel this patch is still needed.<br/> The point of the patch was to test the return value of cfs_cpt_bind() in cfs_wi_scheduler(). The fact that a local variable was removed from this function does not change this.<br/> I rewrote the patch replacing the use of this 'name' local variable with sched-&gt;ws_name.</p> <p>Here is the new patch:<br/> <a href="http://review.whamcloud.com/7103" class="external-link" target="_blank" rel="nofollow noopener">http://review.whamcloud.com/7103</a></p> <p>Please reopen this ticket.</p> <p>Thanks,<br/> Sebastien.</p> <p>Reopened per customer request.</p> <p>Patch has landed to master.</p> Development Rank 1|hzvse7: Rank (Obsolete) 8500