Changeset 1f92232


Ignore:
Timestamp:
Aug 26, 2018 1:50:40 PM (2 years ago)
Author:
zzz <zzz@…>
Branches:
master
Children:
e64ad7c
Parents:
db54edc
Message:

Findbugs all over

Files:
17 edited

Legend:

Unmodified
Added
Removed
  • apps/addressbook/java/src/net/i2p/addressbook/Daemon.java

    rdb54edc r1f92232  
    190190                if (knownNames != null) {
    191191                    oldDest = null;
    192                     isKnown = key != null ? knownNames.contains(key) : null;
     192                    isKnown = key != null ? knownNames.contains(key) : false;
    193193                } else {
    194194                    oldDest = key != null ? router.lookup(key) : null;
     
    804804    public static void main(String[] args) {
    805805        Daemon daemon = new Daemon();
    806         if (args != null && args.length > 0 && args[0].equals("test"))
     806        if (args.length > 0 && args[0].equals("test"))
    807807            daemon.test(args);
    808808        else
     
    824824    }
    825825   
     826    /**
     827     *  @param args may be null
     828     */
    826829    public void run(String[] args) {
    827830        _running = true;
    828831        String settingsLocation = "config.txt";
    829832        File homeFile;
    830         if (args.length > 0) {
     833        if (args != null && args.length > 0) {
    831834            homeFile = new SecureDirectory(args[0]);
    832835            if (!homeFile.isAbsolute())
  • apps/addressbook/java/src/net/metanotion/io/block/BlockFile.java

    rdb54edc r1f92232  
    591591                metaIndex.close();
    592592
    593                 Set<String> oi = openIndices.keySet();
    594                 Iterator<String> i = oi.iterator();
    595                 Object k;
    596                 while(i.hasNext()) {
    597                         k = i.next();
    598                         BSkipList bsl = openIndices.get(k);
     593                for (BSkipList bsl : openIndices.values()) {
    599594                        bsl.close();
    600595                }
  • apps/i2psnark/java/src/org/klomp/snark/SnarkManager.java

    rdb54edc r1f92232  
    202202
    203203    /** host names for config form */
    204     public static final Set<String> KNOWN_OPENTRACKERS = new HashSet<String>(Arrays.asList(new String[] {
     204    static final Set<String> KNOWN_OPENTRACKERS = new HashSet<String>(Arrays.asList(new String[] {
    205205        "tracker.welterde.i2p", "cfmqlafjfmgkzbt4r3jsfyhgsr5abgxryl6fnz3d3y5a365di5aa.b32.i2p",
    206206        "opentracker.dg2.i2p", "w7tpbzncbcocrqtwwm3nezhnnsw4ozadvi2hmvzdhrqzfxfum7wa.b32.i2p",
  • apps/i2ptunnel/java/src/net/i2p/i2ptunnel/ui/TunnelConfig.java

    rdb54edc r1f92232  
    812812        };
    813813
    814     public static final Set<String> _noShowSet = new HashSet<String>(128);
    815     public static final Set<String> _nonProxyNoShowSet = new HashSet<String>(4);
     814    static final Set<String> _noShowSet = new HashSet<String>(128);
     815    static final Set<String> _nonProxyNoShowSet = new HashSet<String>(4);
    816816    static {
    817817        _noShowSet.addAll(Arrays.asList(_noShowOpts));
  • apps/routerconsole/java/src/net/i2p/router/web/helpers/ConfigSummaryHandler.java

    rdb54edc r1f92232  
    103103                    }
    104104                }
    105                 for (Iterator<Integer> iter = sections.keySet().iterator(); iter.hasNext(); ) {
    106                     int i = iter.next();
     105                for (Iterator<Map.Entry<Integer, String>> iter = sections.entrySet().iterator(); iter.hasNext(); ) {
     106                    Map.Entry<Integer, String> e = iter.next();
     107                    Integer i = e.getKey();
    107108                    if (toDelete.contains(i)) {
    108                         String removedName = sections.get(i);
     109                        String removedName = e.getValue();
    109110                        iter.remove();
    110111                        addFormNotice(_t("Removed") + ": " + removedName);
  • apps/sam/java/src/net/i2p/sam/SAMStreamSession.java

    rdb54edc r1f92232  
    2222import java.util.Iterator;
    2323import java.util.List;
     24import java.util.Map;
    2425import java.util.Properties;
    2526import java.util.Set;
     
    397398     */
    398399    protected int createSocketHandler ( I2PSocket s, int id ) {
    399         SAMStreamSessionSocketReader reader = null;
    400         StreamSender sender = null;
     400        SAMStreamSessionSocketReader reader;
     401        StreamSender sender;
    401402        if (id == 0) {
    402403            id = createUniqueId();
     
    469470
    470471        synchronized (handlersMap) {
    471             reader = handlersMap.remove(Integer.valueOf(id));
    472             sender = sendersMap.remove(Integer.valueOf(id));
     472            Integer iid = Integer.valueOf(id);
     473            reader = handlersMap.remove(iid);
     474            sender = sendersMap.remove(iid);
    473475        }
    474476
     
    487489     */
    488490    private void removeAllSocketHandlers() {
    489         Integer id;
    490         Set<Integer> keySet;
    491         Iterator<Integer> iter;
    492 
    493491        synchronized (handlersMap) {
    494             keySet = handlersMap.keySet();
    495             iter = keySet.iterator();
    496            
    497             while (iter.hasNext()) {
    498                  id = iter.next();
    499                  handlersMap.get(id).stopRunning();
     492            for (Map.Entry<Integer, SAMStreamSessionSocketReader> e : handlersMap.entrySet()) {
     493                 Integer id = e.getKey();
     494                 e.getValue().stopRunning();
    500495                 sendersMap.get(id).shutDownGracefully();
    501496            }
  • apps/streaming/java/src/net/i2p/client/streaming/impl/MessageInputStream.java

    rdb54edc r1f92232  
    289289                buf.append(" not ready blocks: [");
    290290                long notAvailable = 0;
    291                 for (Long id : _notYetReadyBlocks.keySet()) {
    292                     ByteArray ba = _notYetReadyBlocks.get(id);
     291                for (Map.Entry<Long, ByteArray> e : _notYetReadyBlocks.entrySet()) {
     292                    Long id = e.getKey();
     293                    ByteArray ba = e.getValue();
    293294                    buf.append(id).append(" ");
    294                    
    295                     if (ba != null)
    296                         notAvailable += ba.getValid();
     295                    notAvailable += ba.getValid();
    297296                }
    298297               
     
    590589                buf.append(" not ready blocks: [");
    591590                long notAvailable = 0;
    592                 for (Long id : _notYetReadyBlocks.keySet()) {
    593                     ByteArray ba = _notYetReadyBlocks.get(id);
     591                for (Map.Entry<Long, ByteArray> e : _notYetReadyBlocks.entrySet()) {
     592                    Long id = e.getKey();
     593                    ByteArray ba = e.getValue();
    594594                    buf.append(id).append(" ");
    595                     if (ba != null)
    596                         notAvailable += ba.getValid();
     595                    notAvailable += ba.getValid();
    597596                }
    598597                buf.append("] not ready bytes: ").append(notAvailable);
  • apps/susidns/src/java/src/i2p/susi/dns/TranslateSVGServlet.java

    rdb54edc r1f92232  
    3232    private static final long serialVersionUID = 18638760L;
    3333    private final I2PAppContext _context;
    34     private final String DIR = "/svg";
     34    private static final String DIR = "/svg";
    3535
    3636    public TranslateSVGServlet() {
  • apps/susimail/src/src/i2p/susi/webmail/WebMail.java

    rdb54edc r1f92232  
    899899                                synchronized(_so) {
    900900                                        mc = _so.caches.get(DIR_FOLDER);
     901                                        if (mc == null) {
     902                                                _so.error += "Internal error, no folder\n";
     903                                                return;
     904                                        }
    901905                                        while (mc.isLoading()) {
    902906                                                try {
     
    911915                                // fetch the mail outside the lock
    912916                                // TODO, would be better to add each email as we get it
    913                                 if (mc != null) {
    914                                         found = mc.getMail(MailCache.FetchMode.HEADER);
    915                                 }
     917                                found = mc.getMail(MailCache.FetchMode.HEADER);
    916918                        }
    917919                        if (log.shouldDebug()) log.debug("CW.FNM connected? " + connected + " found? " + found);
     
    28992901                                        sessionObject.error += relay.error;
    29002902                                }
    2901                                 sessionObject.info.replace(_t("Sending mail.") + '\n', "");
     2903                                sessionObject.info = sessionObject.info.replace(_t("Sending mail.") + '\n', "");
    29022904                        }
    29032905                }
     
    33783380                }
    33793381                out.println(button(LOGOUT, _t("Logout") ));
    3380                 if (mail.hasBody() && !mc.getFolderName().equals(DIR_DRAFTS)) {
     3382                if (hasHeader && mail.hasBody() && !mc.getFolderName().equals(DIR_DRAFTS)) {
    33813383                        // can't move unless has body
    33823384                        // can't move from drafts
  • build.xml

    rdb54edc r1f92232  
    878878                <pathelement location="apps/jetty/jettylib/jetty-xml.jar" />
    879879                <pathelement location="apps/jetty/jettylib/tomcat-api.jar" />
    880                 <pathelement location="apps/jrobin/jrobin-1.5.9.1.jar" />
    881880                <pathelement location="installer/lib/wrapper/all/wrapper.jar" />
    882881                <!-- following are only for debian builds -->
     
    20612060    <target name="findbugs" depends="build2, buildUtilityJar">
    20622061        <echo message="Starting findbugs, this will take a while..." />
     2062        <!-- ref: https://github.com/spotbugs/spotbugs/issues/148 -->
     2063        <echo message="If this doesn't work with Java 9+. try Java 8" />
    20632064        <exec executable="nice" failonerror="true">
    20642065            <arg value="findbugs"/>
     
    20712072            <arg value="i2p.fba"/>
    20722073            <arg value="-auxclasspath"/>
    2073             <arg value="build/commons-el.jar:build/jasper-runtime.jar:build/javax.servlet.jar:build/org.mortbay.jetty.jar:apps/jrobin/jrobin-1.5.9.1.jar:installer/lib/wrapper/all/wrapper.jar:apps/susidns/src/lib/standard.jar:apps/susidns/src/lib/jstl.jar:apps/jrobin/jrobin-1.5.9.1.jar"/>
     2074            <arg value="build/commons-el.jar:build/jasper-runtime.jar:build/javax.servlet.jar:build/org.mortbay.jetty.jar:build/jrobin.jar:installer/lib/wrapper/all/wrapper.jar:apps/susidns/src/lib/standard.jar:apps/susidns/src/lib/jstl.jar"/>
    20742075            <arg value="-sourcepath"/>
    20752076            <arg value="apps/BOB/src/:apps/addressbook/java/src/:apps/i2psnark/java/src/:apps/i2ptunnel/java/src/:apps/ministreaming/java/src/:apps/routerconsole/java/src/:apps/sam/java/src/:apps/streaming/java/src/:apps/susidns/src/java/src/:apps/susimail/src/src/:apps/systray/java/src/:core/java/src/:router/java/src/:installer/java/src"/>
  • core/java/src/net/i2p/crypto/KeyStoreUtil.java

    rdb54edc r1f92232  
    4545public final class KeyStoreUtil {
    4646       
    47     public static boolean _blacklistLogged;
     47    private static boolean _blacklistLogged;
    4848
    4949    public static final String DEFAULT_KEYSTORE_PASSWORD = "changeit";
     
    13321332        String pw = args[3];
    13331333        boolean ok = createKeys(ksf, DEFAULT_KEYSTORE_PASSWORD, alias, "test cname", "test ou",
    1334                                 //DEFAULT_KEY_VALID_DAYS, "EdDSA", 256, pw);
    1335                                 DEFAULT_KEY_VALID_DAYS, "ElGamal", 2048, pw);
     1334                                DEFAULT_KEY_VALID_DAYS, "EdDSA", 256, pw);
     1335                                //DEFAULT_KEY_VALID_DAYS, "ElGamal", 2048, pw);
    13361336        System.out.println("genkey ok? " + ok);
    13371337    }
  • core/java/src/net/i2p/crypto/TrustedUpdate.java

    rdb54edc r1f92232  
    830830     */
    831831    public String verifyAndGetSigner(File signedFile) {
    832         for (SigningPublicKey signingPublicKey : _trustedKeys.keySet()) {
     832        for (Map.Entry<SigningPublicKey, String> e : _trustedKeys.entrySet()) {
     833            SigningPublicKey signingPublicKey = e.getKey();
    833834            boolean isValidSignature = verify(signedFile, signingPublicKey);
    834835            if (isValidSignature)
    835                 return _trustedKeys.get(signingPublicKey);
     836                return e.getValue();
    836837        }
    837838        return null;
  • router/java/src/com/southernstorm/noise/protocol/HandshakeState.java

    rdb54edc r1f92232  
    4242        private DHState remotePublicKey;
    4343        private DHState remoteEphemeral;
    44         private DHState fixedEphemeral;
    4544        private int action;
    4645        private final int requirements;
     
    487486                                                if (localEphemeral == null)
    488487                                                        throw new IllegalStateException("Pattern definition error");
    489                                                 if (fixedEphemeral == null)
    490                                                         localEphemeral.generateKeyPair();
    491                                                 else
    492                                                         localEphemeral.copyFrom(fixedEphemeral);
     488                                                localEphemeral.generateKeyPair();
    493489                                                len = localEphemeral.getPublicKeyLength();
    494490                                                if (space < len)
     
    803799                if (remoteEphemeral != null)
    804800                        remoteEphemeral.destroy();
    805                 if (fixedEphemeral != null)
    806                         fixedEphemeral.destroy();
    807801                if (prologue != null)
    808802                        Noise.destroy(prologue);
  • router/java/src/net/i2p/data/i2np/DeliveryInstructions.java

    rdb54edc r1f92232  
    313313    }
    314314   
     315/****
    315316    private byte[] getAdditionalInfo() {
    316317        int additionalSize = getAdditionalInfoSize();
     
    323324        return rv;
    324325    }
    325 
     326****/
     327
     328    /** */
    326329    private int getAdditionalInfo(byte rv[], int offset) {
    327330        int origOffset = offset;
  • router/java/src/net/i2p/router/PersistentKeyRing.java

    rdb54edc r1f92232  
    4040    public SessionKey remove(Object o) {
    4141        SessionKey rv = super.remove(o);
    42         if (rv != null && o != null && o instanceof Hash) {
     42        if (rv != null && o instanceof Hash) {
    4343            Hash h = (Hash) o;
    4444            _ctx.router().saveConfig(PROP_PFX + h.toBase64().replace("=", "$"), null);
  • router/java/src/net/i2p/router/transport/ntcp/NTCPTransport.java

    rdb54edc r1f92232  
    613613    void sendComplete(OutNetMessage msg) { _finisher.add(msg); }
    614614
    615     private boolean isEstablished(RouterIdentity peer) {
    616         return isEstablished(peer.calculateHash());
    617     }
    618 
    619615    @Override
    620616    public boolean isEstablished(Hash dest) {
  • router/java/src/net/i2p/router/transport/udp/UDPPacket.java

    rdb54edc r1f92232  
    408408     */
    409409    public static UDPPacket acquire(RouterContext ctx, boolean inbound) {
    410         UDPPacket rv = null;
     410        UDPPacket rv;
    411411        if (CACHE) {
    412412            PacketFactory.context = ctx;
    413413            rv = _packetCache.acquire();
    414414            rv.init(ctx);
    415         }
    416         if (rv == null)
     415        } else {
    417416            rv = new UDPPacket(ctx);
     417        }
    418418        return rv;
    419419    }
Note: See TracChangeset for help on using the changeset viewer.