Changeset 002d057


Ignore:
Timestamp:
Jun 13, 2014 9:37:18 PM (7 years ago)
Author:
zzz <zzz@…>
Branches:
master
Children:
8fd2a05b
Parents:
ab44488
Message:

findbugs all over

Files:
19 edited

Legend:

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

    rab44488 r002d057  
    3838  private final PeerCoordinator coordinator;
    3939
    40   private long lastDownloaded = 0;
    41   private long lastUploaded = 0;
     40  //private long lastDownloaded = 0;
     41  //private long lastUploaded = 0;
    4242
    4343  PeerMonitorTask(PeerCoordinator coordinator)
  • apps/i2psnark/java/src/org/klomp/snark/web/I2PSnarkServlet.java

    rab44488 r002d057  
    5050    private String _contextName;
    5151    private SnarkManager _manager;
    52     private static long _nonce;
     52    private long _nonce;
    5353    private String _themePath;
    5454    private String _imgPath;
  • apps/i2ptunnel/java/src/net/i2p/i2ptunnel/I2PTunnel.java

    rab44488 r002d057  
    325325     *  Generic options used for clients and servers.
    326326     *  NOT a copy, Do NOT modify for per-connection options, make a copy.
    327      *  @return NOT a copy, do NOT modify for per-connection options
     327     *  @return non-null, NOT a copy, do NOT modify for per-connection options
    328328     */
    329329    public Properties getClientOptions() { return _clientOptions; }
  • apps/i2ptunnel/java/src/net/i2p/i2ptunnel/I2PTunnelHTTPClientBase.java

    rab44488 r002d057  
    609609        }
    610610        byte[] header = getErrorPage(error, ERR_DESTINATION_UNKNOWN);
    611         String message = ise.getLocalizedMessage();
     611        String message = ise != null ? ise.getLocalizedMessage() : "unknown error";
    612612        try {
    613613            writeErrorMessage(header, message, out, targetRequest, usingWWWProxy, wwwProxy);
  • apps/i2ptunnel/java/src/net/i2p/i2ptunnel/I2PTunnelHTTPServer.java

    rab44488 r002d057  
    256256            String spoofHost;
    257257            int ourPort = socket.getLocalPort();
    258             if (ourPort != 80 && ourPort > 0 && ourPort <= 65535 && opts != null) {
     258            if (ourPort != 80 && ourPort > 0 && ourPort <= 65535) {
    259259                String portSpoof = opts.getProperty("spoofedHost." + ourPort);
    260260                if (portSpoof != null)
     
    285285            // if it was one of the Accept-encoding: values, and gzip the payload       
    286286            boolean allowGZIP = true;
    287             if (opts != null) {
    288                 String val = opts.getProperty("i2ptunnel.gzip");
    289                 if ( (val != null) && (!Boolean.parseBoolean(val)) )
    290                     allowGZIP = false;
    291             }
     287            String val = opts.getProperty("i2ptunnel.gzip");
     288            if ( (val != null) && (!Boolean.parseBoolean(val)) )
     289                allowGZIP = false;
    292290            if (_log.shouldLog(Log.INFO))
    293291                _log.info("HTTP server encoding header: " + enc + "/" + altEnc);
  • apps/routerconsole/java/src/net/i2p/router/update/ConsoleUpdateManager.java

    rab44488 r002d057  
    15811581
    15821582        @Override
     1583        public boolean equals(Object o) {
     1584            return super.equals(o) && (o instanceof VersionAvailable);
     1585        }
     1586
     1587        @Override
    15831588        public String toString() {
    15841589            return "VersionAvailable \"" + version + "\" " + sourceMap +
  • apps/streaming/java/src/net/i2p/client/streaming/impl/PcapWriter.java

    rab44488 r002d057  
    236236        else
    237237            seq = pkt.getSequenceNum();
     238        DataHelper.writeLong(_fos, 4, seq);
    238239        long acked = 0;
    239240        if (con != null) {
    240241            acked = getLowestAckedThrough(pkt, con);
    241242        }
    242         DataHelper.writeLong(_fos, 4, pkt.getSequenceNum());
    243243        DataHelper.writeLong(_fos, 4, acked);
    244244
  • apps/susimail/src/src/i2p/susi/webmail/Mail.java

    rab44488 r002d057  
    5858        public String sender,   // as received, trimmed only, not HTML escaped
    5959                reply, subject, dateString,
    60                 formattedSender,    // address only, enclosed with <>, not HTML escaped
     60                //formattedSender,    // address only, enclosed with <>, not HTML escaped
    6161                formattedSubject,
    6262                formattedDate,  // US Locale, UTC
     
    7979        public Mail(String uidl) {
    8080                this.uidl = uidl;
    81                 formattedSender = unknown;
     81                //formattedSender = unknown;
    8282                formattedSubject = unknown;
    8383                formattedDate = unknown;
     
    307307                                                if( line.startsWith( "From:" ) ) {
    308308                                                        sender = line.substring( 5 ).trim();
    309                                                         formattedSender = getAddress( sender );
     309                                                        //formattedSender = getAddress( sender );
    310310                                                        shortSender = sender.replace("\"", "").trim();
    311311                                                        int lt = shortSender.indexOf('<');
  • apps/susimail/src/src/i2p/susi/webmail/MailPart.java

    rab44488 r002d057  
    114114                                if( str != null )
    115115                                        boundary = str;
    116                                 if( x_type != null && x_type.startsWith( "multipart" ) && boundary != null )
     116                                if (x_type.startsWith( "multipart" ) && boundary != null )
    117117                                        x_multipart = true;
    118                                 if( x_type != null && x_type.startsWith( "message" ) )
     118                                if (x_type.startsWith( "message" ) )
    119119                                        x_message = true;
    120120                                str = getHeaderLineAttribute( headerLines[i], "name" );
  • apps/susimail/src/src/i2p/susi/webmail/pop3/BackgroundChecker.java

    rab44488 r002d057  
    3333        // short for testing
    3434        //private static final long MIN_IDLE = 10*60*1000;
    35         private final long MIN_IDLE = 30*60*1000;
     35        private static final long MIN_IDLE = 30*60*1000;
    3636        // short for testing
    3737        //private static final long MIN_SINCE = 10*60*1000;
    38         private final long MIN_SINCE = 60*60*1000;
     38        private static final long MIN_SINCE = 60*60*1000;
    3939
    4040        public BackgroundChecker(POP3MailBox mailbox) {
  • apps/susimail/src/src/i2p/susi/webmail/pop3/POP3MailBox.java

    rab44488 r002d057  
    137137        private ReadBuffer getHeader( int id ) {
    138138                        Debug.debug(Debug.DEBUG, "getHeader(" + id + ")");
    139                         Integer idObj = Integer.valueOf(id);
    140139                        ReadBuffer header = null;
    141140                        if (id >= 1 && id <= mails) {
     
    235234        private ReadBuffer getBody(int id) {
    236235                        Debug.debug(Debug.DEBUG, "getBody(" + id + ")");
    237                         Integer idObj = Integer.valueOf(id);
    238236                        ReadBuffer body = null;
    239237                        if (id >= 1 && id <= mails) {
  • core/java/src/net/i2p/client/RequestLeaseSetMessageHandler.java

    rab44488 r002d057  
    118118                spk = new SigningPrivateKey(dummy);
    119119            }
    120             session.getProducer().createLeaseSet(session, leaseSet, li.getSigningPrivateKey(), li.getPrivateKey());
     120            session.getProducer().createLeaseSet(session, leaseSet, spk, li.getPrivateKey());
    121121            session.setLeaseSet(leaseSet);
    122122        } catch (DataFormatException dfe) {
  • core/java/src/net/i2p/data/Destination.java

    rab44488 r002d057  
    182182        }
    183183    }
     184
     185    @Override
     186    public boolean equals(Object o) {
     187        return super.equals(o) && (o instanceof Destination);
     188    }
    184189}
  • core/java/src/net/i2p/data/RouterIdentity.java

    rab44488 r002d057  
    3030        return (_certificate != null) && (_certificate.getCertificateType() == Certificate.CERTIFICATE_TYPE_HIDDEN);
    3131    }
     32
     33    @Override
     34    public boolean equals(Object o) {
     35        return super.equals(o) && (o instanceof LeaseSet);
     36    }
    3237}
  • core/java/src/net/i2p/data/i2cp/SessionConfig.java

    rab44488 r002d057  
    204204    }
    205205
    206     /* FIXME missing hashCode() method FIXME */
    207206    @Override
    208207    public boolean equals(Object object) {
     
    216215         
    217216        return false;
     217    }
     218
     219    @Override
     220    public int hashCode() {
     221        return _signature != null ? _signature.hashCode() : 0;
    218222    }
    219223
  • core/java/src/net/i2p/util/LogRecord.java

    rab44488 r002d057  
    8181                (_throwable != null && r._throwable != null && _throwable.getClass() == r._throwable.getClass()));
    8282    }
     83
     84    @Override
     85    public int hashCode() {
     86        return _message != null ? _message.hashCode() : 0;
     87    }
    8388}
  • core/java/src/net/i2p/util/NativeBigInteger.java

    rab44488 r002d057  
    806806        return "jbigi-linux-";
    807807    }
     808
     809    @Override
     810    public boolean equals(Object o) {
     811        // for findbugs
     812        return super.equals(o);
     813    }
    808814}
  • router/java/src/net/i2p/router/message/OutboundClientMessageJobHelper.java

    rab44488 r002d057  
    261261        // garlic encryption without any ElGamal (yay)
    262262        MessageWrapper.OneTimeSession sess = MessageWrapper.generateSession(ctx, skm);
    263         if (sess == null)
    264             return null;
    265263        GarlicMessage msg = MessageWrapper.wrap(ctx, dsm, sess);
    266264        return msg;
  • router/java/src/net/i2p/router/networkdb/kademlia/MessageWrapper.java

    rab44488 r002d057  
    188188     *  avoiding ElGamal.
    189189     *
     190     *  @return non-null
    190191     *  @since 0.9.9
    191192     */
Note: See TracChangeset for help on using the changeset viewer.