View Issue Details

IDProjectCategoryView StatusLast Update
0003903unrealircdpublic2010-04-21 21:37
ReporterAdam- Assigned To 
PrioritynormalSeverityminorReproducibilityalways
Status closedResolutionno change required 
Product Version3.2.8 
Summary0003903: ULines affected by servers SendQ limit
DescriptionULined servers are affected by servers SendQ limit:

!irc.localhost.com *** Notice -- Max SendQ limit exceeded for
          services.localhost.net[192.168.1.2]: 1000071 > 1000000
!irc.localhost.com *** Global -- Closing link: Max SendQ exceeded -
          services.localhost.net[192.168.1.2]

/stats U:
21:15 -!- U services.localhost.net
21:15 -!- U End of /STATS report

Relevant parts of config:

class servers
{
        pingfreq 90;
        maxclients 10; /* Max servers we can have linked at a time */
        sendq 1000000;
        connfreq 100; /* How many seconds between each connection attempt */
};

link services.localhost.net
{
        username *;
        hostname *;
        bind-ip *;
        port 6667;
        hub *;
        password-connect "mypass";
        password-receive "mypass";
        class servers;
};

ulines {
        services.localhost.net;
};
Steps To ReproduceULine a server and exceed the SendQ
3rd party modules

Activities

syzop

2010-04-21 21:37

administrator   ~0016077

Last edited: 2010-04-21 21:41

Yes, that is correct. If that's a problem, then put your services in a separate class :)

Note that sendq also exists to provide some safety. If services would go nuts, or be lagged as hell, then this can cause a big pile of data being built up (queued). Not obeying sendq can therefore lead to memory exhaustion (granted, that would be a pretty bad mem limit and pretty bad flood/lag).

Personally, I always put services in their own class anyway, but hey.. that's just me :P.

*EDIT: totally reworded*

Issue History

Date Modified Username Field Change
2010-04-21 03:43 Adam- New Issue
2010-04-21 21:37 syzop QA => Not touched yet by developer
2010-04-21 21:37 syzop U4: Need for upstream patch => No need for upstream InspIRCd patch
2010-04-21 21:37 syzop Note Added: 0016077
2010-04-21 21:37 syzop Status new => closed
2010-04-21 21:37 syzop Resolution open => no change required
2010-04-21 21:41 syzop Note Edited: 0016077