findbugsExcludeFile.xml 7.8 KB

123456789101112131415161718192021222324252627282930313233343536373839404142434445464748495051525354555657585960616263646566676869707172737475767778798081828384858687888990919293949596979899100101102103104105106107108109110111112113114115116117118119120121122123124125126127128129130131132133134135136137138139140141142143144145146147148149150151152153154155156157158159160161162163164165166167168169170171172173174175176177178179180181182183184185186187188189190191192193194195196197198199200201202203204205206207208209210211212213214215216217218219220221222223224225226227228229230
  1. <FindBugsFilter>
  2. <Match>
  3. <Package name="org.apache.hadoop.record.compiler.generated" />
  4. </Match>
  5. <Match>
  6. <Package name="org.apache.hadoop.hdfs.protocol.proto" />
  7. </Match>
  8. <Match>
  9. <Package name="org.apache.hadoop.hdfs.server.namenode.ha.proto" />
  10. </Match>
  11. <Match>
  12. <Class name="~org.apache.hadoop.hdfs.server.namenode.FsImageProto.*" />
  13. </Match>
  14. <Match>
  15. <Package name="org.apache.hadoop.hdfs.qjournal.protocol" />
  16. </Match>
  17. <Match>
  18. <Bug pattern="EI_EXPOSE_REP" />
  19. </Match>
  20. <Match>
  21. <Bug pattern="EI_EXPOSE_REP2" />
  22. </Match>
  23. <Match>
  24. <Bug pattern="SE_COMPARATOR_SHOULD_BE_SERIALIZABLE" />
  25. </Match>
  26. <Match>
  27. <Bug pattern="SE_BAD_FIELD" />
  28. </Match>
  29. <Match>
  30. <Class name="~.*_jsp" />
  31. <Bug pattern="DLS_DEAD_LOCAL_STORE" />
  32. </Match>
  33. <Match>
  34. <Field name="_jspx_dependants" />
  35. <Bug pattern="UWF_UNWRITTEN_FIELD" />
  36. </Match>
  37. <!--
  38. Inconsistent synchronization for Client.Connection.out is
  39. is intentional to make a connection to be closed instantly.
  40. -->
  41. <Match>
  42. <Class name="org.apache.hadoop.ipc.Client$Connection" />
  43. <Field name="out" />
  44. <Bug pattern="IS2_INCONSISTENT_SYNC" />
  45. </Match>
  46. <!--
  47. Ignore Cross Scripting Vulnerabilities
  48. We have an input quoting filter that protects us.
  49. -->
  50. <Match>
  51. <Bug code="XSS" />
  52. </Match>
  53. <Match>
  54. <Bug code="HRS" />
  55. </Match>
  56. <!--
  57. core changes
  58. -->
  59. <Match>
  60. <Class name="~org.apache.hadoop.*" />
  61. <Bug code="MS" />
  62. </Match>
  63. <!--
  64. getTmpInputStreams is pretty much like a stream constructor.
  65. The newly created streams are not supposed to be closed in the constructor. So ignore
  66. the OBL warning.
  67. -->
  68. <Match>
  69. <Class name="org.apache.hadoop.hdfs.server.datanode.fsdataset.impl.FsDatasetImpl" />
  70. <Method name="getTmpInputStreams" />
  71. <Bug pattern="OBL_UNSATISFIED_OBLIGATION" />
  72. </Match>
  73. <!--
  74. ResponseProccessor is thread that is designed to catch RuntimeException.
  75. -->
  76. <Match>
  77. <Class name="org.apache.hadoop.hdfs.DFSOutputStream$DataStreamer$ResponseProcessor" />
  78. <Method name="run" />
  79. <Bug pattern="REC_CATCH_EXCEPTION" />
  80. </Match>
  81. <!--
  82. we don't need to synchronize out in newStreamForAppend method because
  83. out.streamer cannot be touched by other threads. It is false-positive.
  84. -->
  85. <Match>
  86. <Class name="org.apache.hadoop.hdfs.DFSOutputStream" />
  87. <Method name="newStreamForAppend" />
  88. <Field name="streamer" />
  89. <Bug pattern="IS2_INCONSISTENT_SYNC" />
  90. </Match>
  91. <!--
  92. lastAppliedTxid is carefully unsynchronized in the BackupNode in a couple spots.
  93. See the comments in BackupImage for justification.
  94. -->
  95. <Match>
  96. <Class name="org.apache.hadoop.hdfs.server.namenode.FSImage" />
  97. <Field name="lastAppliedTxId" />
  98. <Bug pattern="IS2_INCONSISTENT_SYNC" />
  99. </Match>
  100. <!--
  101. Findbugs doesn't realize that closing a FilterOutputStream pushes the close down to
  102. wrapped streams, too.
  103. -->
  104. <Match>
  105. <Class name="org.apache.hadoop.hdfs.server.namenode.FSImageFormat$Saver" />
  106. <Method name="save" />
  107. <Bug pattern="OS_OPEN_STREAM" />
  108. </Match>
  109. <!--
  110. the 'metrics' member is sometimes used from synchronized blocks, sometimes not,
  111. but it's only reset by test cases, so should be fine
  112. -->
  113. <Match>
  114. <Class name="org.apache.hadoop.hdfs.server.namenode.FSEditLog" />
  115. <Field name="metrics" />
  116. <Bug pattern="IS2_INCONSISTENT_SYNC" />
  117. </Match>
  118. <!--
  119. We use a separate lock to protect modifications to journalSet so that
  120. FSEditLog#selectInputStreams does not need to be a synchronized method.
  121. -->
  122. <Match>
  123. <Class name="org.apache.hadoop.hdfs.server.namenode.FSEditLog" />
  124. <Field name="journalSet" />
  125. <Bug pattern="IS2_INCONSISTENT_SYNC" />
  126. </Match>
  127. <!--
  128. This method isn't performance-critical and is much clearer to write as it's written.
  129. -->
  130. <Match>
  131. <Class name="org.apache.hadoop.hdfs.server.datanode.BlockPoolManager" />
  132. <Method name="doRefreshNamenodes" />
  133. <Bug category="PERFORMANCE" />
  134. </Match>
  135. <!-- Don't complain about System.exit() being called from quit() -->
  136. <Match>
  137. <Class name="org.apache.hadoop.hdfs.server.namenode.MetaRecoveryContext" />
  138. <Method name="quit" />
  139. <Bug pattern="DM_EXIT" />
  140. </Match>
  141. <!-- Don't complain about recoverBlock equality check -->
  142. <Match>
  143. <Class name="org.apache.hadoop.hdfs.server.datanode.DataNode" />
  144. <Method name="recoverBlock" />
  145. <Bug pattern="EC_UNRELATED_TYPES" />
  146. </Match>
  147. <!-- More complex cleanup logic confuses findbugs -->
  148. <Match>
  149. <Class name="org.apache.hadoop.hdfs.qjournal.server.Journal" />
  150. <Method name="persistPaxosData" />
  151. <Bug pattern="OS_OPEN_STREAM" />
  152. </Match>
  153. <!-- Don't complain about LocalDatanodeInfo's anonymous class -->
  154. <Match>
  155. <Class name="org.apache.hadoop.hdfs.BlockReaderLocal$LocalDatanodeInfo$1" />
  156. <Bug pattern="SE_BAD_FIELD_INNER_CLASS" />
  157. </Match>
  158. <!-- Only one method increments numFailedVolumes and it is synchronized -->
  159. <Match>
  160. <Class name="org.apache.hadoop.hdfs.server.datanode.fsdataset.impl.FsVolumeList" />
  161. <Field name="numFailedVolumes" />
  162. <Bug pattern="VO_VOLATILE_INCREMENT" />
  163. </Match>
  164. <!-- Access to pendingReceivedRequests is synchronized -->
  165. <Match>
  166. <Class name="org.apache.hadoop.hdfs.server.datanode.BPServiceActor" />
  167. <Method name="notifyNamenodeBlockImmediately" />
  168. <Field name="pendingReceivedRequests" />
  169. <Bug pattern="VO_VOLATILE_INCREMENT" />
  170. </Match>
  171. <!-- Replace System.exit() call with ExitUtil.terminate() -->
  172. <Match>
  173. <Class name="org.apache.hadoop.hdfs.tools.JMXGet"/>
  174. <Method name="main" />
  175. <Bug pattern="NP_NULL_ON_SOME_PATH" />
  176. </Match>
  177. <Match>
  178. <Class name="org.apache.hadoop.hdfs.server.datanode.ReplicaInfo" />
  179. <Method name="setDirInternal" />
  180. <Bug pattern="DM_STRING_CTOR" />
  181. </Match>
  182. <Match>
  183. <Class name="org.apache.hadoop.hdfs.shortcircuit.DfsClientShmManager$EndpointShmManager" />
  184. <Method name="allocSlot" />
  185. <Bug pattern="UL_UNRELEASED_LOCK_EXCEPTION_PATH" />
  186. </Match>
  187. <Match>
  188. <Class name="org.apache.hadoop.hdfs.shortcircuit.DfsClientShmManager$EndpointShmManager" />
  189. <Method name="allocSlot" />
  190. <Bug pattern="UL_UNRELEASED_LOCK" />
  191. </Match>
  192. <!-- Manually verified to be okay, we want to throw away the top bit here -->
  193. <Match>
  194. <Class name="org.apache.hadoop.hdfs.server.namenode.CachedBlock" />
  195. <Method name="getReplication" />
  196. <Bug pattern="ICAST_QUESTIONABLE_UNSIGNED_RIGHT_SHIFT" />
  197. </Match>
  198. <Match>
  199. <Class name="org.apache.hadoop.hdfs.protocol.CacheDirective" />
  200. <Method name="insertInternal" />
  201. <Bug pattern="BC_UNCONFIRMED_CAST" />
  202. </Match>
  203. <Match>
  204. <Class name="org.apache.hadoop.hdfs.server.blockmanagement.CacheReplicationMonitor" />
  205. <Bug pattern="RV_RETURN_VALUE_IGNORED_BAD_PRACTICE" />
  206. </Match>
  207. <Match>
  208. <Class name="org.apache.hadoop.hdfs.DFSUtil"/>
  209. <Method name="assertAllResultsEqual" />
  210. <Bug pattern="NP_LOAD_OF_KNOWN_NULL_VALUE" />
  211. </Match>
  212. <!--
  213. We use a separate lock to guard cachingStrategy in order to separate
  214. locks for p-reads from seek + read invocations.
  215. -->
  216. <Match>
  217. <Class name="org.apache.hadoop.hdfs.DFSInputStream" />
  218. <Field name="cachingStrategy" />
  219. <Bug pattern="IS2_INCONSISTENT_SYNC" />
  220. </Match>
  221. </FindBugsFilter>