findbugsExcludeFile.xml 7.2 KB

123456789101112131415161718192021222324252627282930313233343536373839404142434445464748495051525354555657585960616263646566676869707172737475767778798081828384858687888990919293949596979899100101102103104105106107108109110111112113114115116117118119120121122123124125126127128129130131132133134135136137138139140141142143144145146147148149150151152153154155156157158159160161162163164165166167168169170171172173174175176177178179180181182183184185186187188189190191192193194195196197198199200201202203204205206207208209210
  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. lastAppliedTxid is carefully unsynchronized in the BackupNode in a couple spots.
  75. See the comments in BackupImage for justification.
  76. -->
  77. <Match>
  78. <Class name="org.apache.hadoop.hdfs.server.namenode.FSImage" />
  79. <Field name="lastAppliedTxId" />
  80. <Bug pattern="IS2_INCONSISTENT_SYNC" />
  81. </Match>
  82. <!--
  83. Findbugs doesn't realize that closing a FilterOutputStream pushes the close down to
  84. wrapped streams, too.
  85. -->
  86. <Match>
  87. <Class name="org.apache.hadoop.hdfs.server.namenode.FSImageFormat$Saver" />
  88. <Method name="save" />
  89. <Bug pattern="OS_OPEN_STREAM" />
  90. </Match>
  91. <!--
  92. the 'metrics' member is sometimes used from synchronized blocks, sometimes not,
  93. but it's only reset by test cases, so should be fine
  94. -->
  95. <Match>
  96. <Class name="org.apache.hadoop.hdfs.server.namenode.FSEditLog" />
  97. <Field name="metrics" />
  98. <Bug pattern="IS2_INCONSISTENT_SYNC" />
  99. </Match>
  100. <!--
  101. We use a separate lock to protect modifications to journalSet so that
  102. FSEditLog#selectInputStreams does not need to be a synchronized method.
  103. -->
  104. <Match>
  105. <Class name="org.apache.hadoop.hdfs.server.namenode.FSEditLog" />
  106. <Field name="journalSet" />
  107. <Bug pattern="IS2_INCONSISTENT_SYNC" />
  108. </Match>
  109. <!--
  110. FSEditLog#getTotalSyncCount is not synchronized because this method is
  111. used by metrics. NullPointerException can happen and it is ignored.
  112. -->
  113. <Match>
  114. <Class name="org.apache.hadoop.hdfs.server.namenode.FSEditLog" />
  115. <Field name="editLogStream" />
  116. <Bug pattern="IS2_INCONSISTENT_SYNC" />
  117. </Match>
  118. <!--
  119. FSEditLog#isOpenForWriteWithoutLock and FSEditLog#isSegmentOpenWithoutLock
  120. are not synchronized because these methods are used by metrics.
  121. -->
  122. <Match>
  123. <Class name="org.apache.hadoop.hdfs.server.namenode.FSEditLog" />
  124. <Field name="state" />
  125. <Bug pattern="IS2_INCONSISTENT_SYNC" />
  126. </Match>
  127. <!--
  128. All of the threads which update/increment txid are synchronized,
  129. so make txid volatile instead of AtomicLong.
  130. -->
  131. <Match>
  132. <Class name="org.apache.hadoop.hdfs.server.namenode.FSEditLog" />
  133. <Field name="txid" />
  134. <Bug pattern="VO_VOLATILE_INCREMENT" />
  135. </Match>
  136. <!--
  137. This method isn't performance-critical and is much clearer to write as it's written.
  138. -->
  139. <Match>
  140. <Class name="org.apache.hadoop.hdfs.server.datanode.BlockPoolManager" />
  141. <Method name="doRefreshNamenodes" />
  142. <Bug category="PERFORMANCE" />
  143. </Match>
  144. <!-- Don't complain about System.exit() being called from quit() -->
  145. <Match>
  146. <Class name="org.apache.hadoop.hdfs.server.namenode.MetaRecoveryContext" />
  147. <Method name="quit" />
  148. <Bug pattern="DM_EXIT" />
  149. </Match>
  150. <!-- More complex cleanup logic confuses findbugs -->
  151. <Match>
  152. <Class name="org.apache.hadoop.hdfs.qjournal.server.Journal" />
  153. <Method name="persistPaxosData" />
  154. <Bug pattern="OS_OPEN_STREAM" />
  155. </Match>
  156. <!-- Don't complain about LocalDatanodeInfo's anonymous class -->
  157. <Match>
  158. <Class name="org.apache.hadoop.hdfs.client.impl.BlockReaderLocal$LocalDatanodeInfo$1" />
  159. <Bug pattern="SE_BAD_FIELD_INNER_CLASS" />
  160. </Match>
  161. <!-- Only one method increments numFailedVolumes and it is synchronized -->
  162. <Match>
  163. <Class name="org.apache.hadoop.hdfs.server.datanode.fsdataset.impl.FsVolumeList" />
  164. <Field name="numFailedVolumes" />
  165. <Bug pattern="VO_VOLATILE_INCREMENT" />
  166. </Match>
  167. <!-- Access to pendingReceivedRequests is synchronized -->
  168. <Match>
  169. <Class name="org.apache.hadoop.hdfs.server.datanode.BPServiceActor" />
  170. <Method name="notifyNamenodeBlockImmediately" />
  171. <Field name="pendingReceivedRequests" />
  172. <Bug pattern="VO_VOLATILE_INCREMENT" />
  173. </Match>
  174. <!-- Replace System.exit() call with ExitUtil.terminate() -->
  175. <Match>
  176. <Class name="org.apache.hadoop.hdfs.tools.JMXGet"/>
  177. <Method name="main" />
  178. <Bug pattern="NP_NULL_ON_SOME_PATH" />
  179. </Match>
  180. <Match>
  181. <Class name="org.apache.hadoop.hdfs.server.datanode.ReplicaInfo" />
  182. <Method name="setDirInternal" />
  183. <Bug pattern="DM_STRING_CTOR" />
  184. </Match>
  185. <!-- Manually verified to be okay, we want to throw away the top bit here -->
  186. <Match>
  187. <Class name="org.apache.hadoop.hdfs.server.namenode.CachedBlock" />
  188. <Method name="getReplication" />
  189. <Bug pattern="ICAST_QUESTIONABLE_UNSIGNED_RIGHT_SHIFT" />
  190. </Match>
  191. <Match>
  192. <Class name="org.apache.hadoop.hdfs.protocol.CacheDirective" />
  193. <Method name="insertInternal" />
  194. <Bug pattern="BC_UNCONFIRMED_CAST" />
  195. </Match>
  196. <Match>
  197. <Class name="org.apache.hadoop.hdfs.server.blockmanagement.CacheReplicationMonitor" />
  198. <Bug pattern="RV_RETURN_VALUE_IGNORED_BAD_PRACTICE" />
  199. </Match>
  200. <Match>
  201. <Class name="org.apache.hadoop.hdfs.DFSUtil"/>
  202. <Method name="assertAllResultsEqual" />
  203. <Bug pattern="NP_LOAD_OF_KNOWN_NULL_VALUE" />
  204. </Match>
  205. </FindBugsFilter>