ntrack can get into endless poll loop if no backend modules found

Bug #785119 reported by Alex Merry
8
This bug affects 1 person
Affects Status Importance Assigned to Milestone
ntrack
Fix Released
Medium
Alexander Sack

Bug Description

Note: this issue actually occurs for me in the context of kded4.

The attached sample program is a minimal test of the problem. Note that the problem only occurs if no backend modules can be found for ntrack.

If no backend modules can be found for ntrack, it uses the noop routines. This means that fd 0 is used for monitoring in QNtrack (because the calloc'd _ntrack_monitor_arch->data element is reinterpreted as an fd).

In the context of kded4, this happens to correspond to an open fd where the other side has been closed, so polling it always returns (I've simulated that in the test app with a pipe where I close the write end).

This means that polling fd 0 returns immediately with POLLHUP, ntrack's no-op routines are triggered (which do nothing), and control returns to the event loop where fd 0 is included in the poll() again, and the cycle continues.

Potential solution: create a dummy pipe (keeping both ends open) in the no-op routines, and return the read end of that.

Revision history for this message
Alex Merry (randomguy3) wrote :
Revision history for this message
Alex Merry (randomguy3) wrote :

This patch fixes the issue in the way I suggested (using a pipe).

Revision history for this message
Alexander Sack (asac) wrote :

question: why are no backend modules found?

otherwise, i posted a patch here on the infinite loop thing: https://bugs.launchpad.net/ntrack/+bug/755608/comments/19

Revision history for this message
Alexander Sack (asac) wrote :

anyway, this patch looks interesting and I might consider to take it for next release ...

I guess you don't have a problem with me relicensing this to BSD/MIT or LGPL-2 (i am considering licensing atm) if it comes to that?

Revision history for this message
Alex Merry (randomguy3) wrote :

Because of this: https://bugs.launchpad.net/ntrack/+bug/785153

The issue I describe here is actually completely different to that in bug 755608 - they just both happen to occur in the context of kded4. Certainly, revision 312 doesn't fix it, although it does apparently fix bug 755608. And I'm not getting an infinite loop within any method in libntrack; it's just that poll() never blocks in the main loop.

Revision history for this message
Alex Merry (randomguy3) wrote :

Nope, I hereby give you permission to use whatever license you wish for any patch I post on launchpad for ntrack.

Revision history for this message
Alexander Sack (asac) wrote :

took the patch slightly modified to ensure that the write end of the pipe is not used as a read fd by frontends.

------------------------------------------------------------
revno: 324
committer: Alexander Sack <email address hidden>
branch nick: ntrack
timestamp: Sun 2011-10-16 02:20:12 +0200
message:
  common: fix poll problems for noop monitor; use pipe to get valid rfds; thx to Alex Merry (randomguy3) lp:785119
------------------------------------------------------------

Changed in ntrack:
status: New → Fix Committed
assignee: nobody → Alexander Sack (asac)
importance: Undecided → Medium
milestone: none → 015
Revision history for this message
Alexander Sack (asac) wrote :

testing this I had to add a follow up patch to fix crashes in monitor recycle test. Point is that API hands our a copy of the rfds buffer and not the buffer itself.

------------------------------------------------------------
revno: 326
committer: Alexander Sack <email address hidden>
branch nick: ntrack
timestamp: Sun 2011-10-16 17:02:53 +0200
message:
  common: fix noop monitor crash caused by double free of pipe fd buffer; hand out copy of rfds
------------------------------------------------------------

Revision history for this message
Alexander Sack (asac) wrote :

ntrack 015 is available including the fix for this bug: https://launchpad.net/ntrack/main/015

Changed in ntrack:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.