Changeset ae76a6e


Ignore:
Timestamp:
Nov 7, 2013 7:20:25 PM (7 years ago)
Author:
zzz <zzz@…>
Branches:
master
Children:
919ec3a, ca5a301
Parents:
5cbecb3
Message:

findbugs correctness fixes, all over the place

Files:
8 edited

Legend:

Unmodified
Added
Removed
  • apps/i2psnark/java/src/org/klomp/snark/web/I2PSnarkServlet.java

    r5cbecb3 rae76a6e  
    24882488                    try {
    24892489                        File f = item;
    2490                         if (f != null) {
    24912490                            long remaining = storage.remaining(f.getCanonicalPath());
    24922491                            if (remaining < 0) {
     
    25082507                                         " (" + DataHelper.formatSize2(remaining) + "B " + _("remaining") + ")";
    25092508                            }
    2510                         } else {
    2511                             status = "Not a file?";
    2512                         }
    25132509                    } catch (IOException ioe) {
    25142510                        status = "Not a file? " + ioe;
  • apps/i2ptunnel/java/src/net/i2p/i2ptunnel/I2PTunnelClientBase.java

    r5cbecb3 rae76a6e  
    319319                if (_log.shouldLog(Log.INFO))
    320320                    _log.info(tunnel.getClientOptions().getProperty("inbound.nickname") + ": Building a new socket manager since the old one closed [s=" + s + "]");
    321                 if (s != null)
    322                     tunnel.removeSession(s);
     321                tunnel.removeSession(s);
    323322                // We could be here a LONG time, holding the lock
    324323                socketManager = buildSocketManager(tunnel, pkf);
  • apps/i2ptunnel/java/src/net/i2p/i2ptunnel/I2PTunnelHTTPServer.java

    r5cbecb3 rae76a6e  
    129129        int pt = getIntOption(OPT_POST_TOTAL_MAX, 0);
    130130        synchronized(this) {
    131             if (pp != 0 || pp != 0 || _postThrottler != null) {
     131            if (pp != 0 || pt != 0 || _postThrottler != null) {
    132132                long pw = 1000L * getIntOption(OPT_POST_WINDOW, DEFAULT_POST_WINDOW);
    133133                long pb = 1000L * getIntOption(OPT_POST_BAN_TIME, DEFAULT_POST_BAN_TIME);
  • apps/routerconsole/java/src/net/i2p/router/web/StatSummarizer.java

    r5cbecb3 rae76a6e  
    44import java.io.IOException;
    55import java.io.OutputStream;
     6import java.util.Collections;
    67import java.util.List;
    78import java.util.HashSet;
     
    310311     */
    311312    Set<Rate> parseSpecs(String specs) {
     313        if (specs == null)
     314            return Collections.EMPTY_SET;
    312315        StringTokenizer tok = new StringTokenizer(specs, ",");
    313316        Set<Rate> rv = new HashSet();
  • apps/streaming/java/src/net/i2p/client/streaming/ConnectionPacketHandler.java

    r5cbecb3 rae76a6e  
    203203                if (_log.shouldLog(Log.WARN))
    204204                    _log.warn(String.format("%s congestion.. dup packet %s ackDelay %d lastSend %d ago",
    205                                     con, packet, now, ackDelay, DataHelper.formatDuration(now - lastSendTime)));
     205                                    con, packet, ackDelay, DataHelper.formatDuration(now - lastSendTime)));
    206206               
    207207                final long nextSendTime = lastSendTime + ackDelay;
  • router/java/src/net/i2p/router/TunnelPoolSettings.java

    r5cbecb3 rae76a6e  
    198198    public Properties getUnknownOptions() { return _unknownOptions; }
    199199   
     200    /**
     201     *  @param prefix non-null
     202     */
    200203    public void readFromProperties(String prefix, Map<Object, Object> props) {
    201204        for (Map.Entry e : props.entrySet()) {
     
    226229                    _priority = Math.min(max, Math.max(MIN_PRIORITY, getInt(value, def)));
    227230                } else
    228                     _unknownOptions.setProperty(name.substring((prefix != null ? prefix.length() : 0)), value);
     231                    _unknownOptions.setProperty(name.substring(prefix.length()), value);
    229232            }
    230233        }
    231         }
    232    
     234    }
     235   
     236    /**
     237     *  @param prefix non-null
     238     */
    233239    public void writeToProperties(String prefix, Properties props) {
    234240        if (props == null) return;
  • router/java/src/net/i2p/router/tunnel/pool/BuildExecutor.java

    r5cbecb3 rae76a6e  
    321321                    // non-zero-hop tunnels
    322322                    // try to kickstart it to build a fallback, otherwise we may get stuck here for a long time (minutes)
    323                     if (mgr.getFreeTunnelCount() <= 0)
    324                         mgr.selectInboundTunnel();
    325                     if (mgr.getOutboundTunnelCount() <= 0)
    326                         mgr.selectOutboundTunnel();
     323                    if (mgr != null) {
     324                        if (mgr.getFreeTunnelCount() <= 0)
     325                            mgr.selectInboundTunnel();
     326                        if (mgr.getOutboundTunnelCount() <= 0)
     327                            mgr.selectOutboundTunnel();
     328                    }
    327329                    synchronized (_currentlyBuilding) {
    328330                        if (!_repoll) {
  • router/java/src/net/i2p/router/tunnel/pool/TunnelPool.java

    r5cbecb3 rae76a6e  
    262262        }
    263263        if (rv != null) {
    264             _context.statManager().addRateData("tunnel.matchLease", closestTo.equals(rv) ? 1 : 0);
     264            _context.statManager().addRateData("tunnel.matchLease", closestTo.equals(rv.getFarEnd()) ? 1 : 0);
    265265        } else {
    266266            if (_log.shouldLog(Log.WARN))
Note: See TracChangeset for help on using the changeset viewer.