Changeset e96cc09


Ignore:
Timestamp:
Oct 10, 2014 3:21:10 PM (7 years ago)
Author:
zzz <zzz@…>
Branches:
master
Children:
e081f94
Parents:
d87178f
Message:

Banlist: Remove unused banlist tracking in the profile causing deadlock (ticket #1394)

Location:
router/java/src/net/i2p/router
Files:
2 edited

Legend:

Unmodified
Added
Removed
  • router/java/src/net/i2p/router/Banlist.java

    rd87178f re96cc09  
    8484            } catch (IllegalStateException ise) {} // next time...
    8585            for (Hash peer : _toUnbanlist) {
    86                 PeerProfile prof = _context.profileOrganizer().getProfile(peer);
    87                 if (prof != null)
    88                     prof.unbanlist();
     86                //PeerProfile prof = _context.profileOrganizer().getProfile(peer);
     87                //if (prof != null)
     88                //    prof.unbanlist();
    8989                _context.messageHistory().unbanlist(peer);
    9090                if (_log.shouldLog(Log.INFO))
     
    156156        } else {
    157157            long period = BANLIST_DURATION_MS + _context.random().nextLong(BANLIST_DURATION_MS / 4);
    158             PeerProfile prof = _context.profileOrganizer().getProfile(peer);
    159             if (prof != null) {
    160                 period = BANLIST_DURATION_MS << prof.incrementBanlists();
    161                 period += _context.random().nextLong(period);
    162             }
     158            //PeerProfile prof = _context.profileOrganizer().getProfile(peer);
     159            //if (prof != null) {
     160            //    period = BANLIST_DURATION_MS << prof.incrementBanlists();
     161            //    period += _context.random().nextLong(period);
     162            //}
    163163       
    164164            if (period > BANLIST_DURATION_MAX)
     
    235235
    236236        if (fully) {
    237             if (realUnbanlist) {
    238                 PeerProfile prof = _context.profileOrganizer().getProfile(peer);
    239                 if (prof != null)
    240                     prof.unbanlist();
    241             }
     237            //if (realUnbanlist) {
     238            //    PeerProfile prof = _context.profileOrganizer().getProfile(peer);
     239            //    if (prof != null)
     240            //        prof.unbanlist();
     241            //}
    242242            _context.messageHistory().unbanlist(peer);
    243243            if (_log.shouldLog(Log.INFO) && e != null)
     
    267267       
    268268        if (unbanlist) {
    269             PeerProfile prof = _context.profileOrganizer().getProfile(peer);
    270             if (prof != null)
    271                 prof.unbanlist();
     269            //PeerProfile prof = _context.profileOrganizer().getProfile(peer);
     270            //if (prof != null)
     271            //    prof.unbanlist();
    272272            _context.messageHistory().unbanlist(peer);
    273273            if (_log.shouldLog(Log.INFO))
  • router/java/src/net/i2p/router/peermanager/PeerProfile.java

    rd87178f re96cc09  
    6666    private boolean _expanded;
    6767    private boolean _expandedDB;
    68     private int _consecutiveBanlists;
     68    //private int _consecutiveBanlists;
    6969    private final int _distance;
    7070   
     
    121121    public boolean getIsExpandedDB() { return _expandedDB; }
    122122   
    123     public int incrementBanlists() { return _consecutiveBanlists++; }
    124     public void unbanlist() { _consecutiveBanlists = 0; }
     123    //public int incrementBanlists() { return _consecutiveBanlists++; }
     124    //public void unbanlist() { _consecutiveBanlists = 0; }
    125125   
    126126    /**
Note: See TracChangeset for help on using the changeset viewer.