cups-browsed uses GMainLoop and global variables, how to introduce locks against race conditions?
Till Kamppeter
till.kamppeter at gmail.com
Fri Dec 2 14:58:41 UTC 2016
Hi,
cups-browsed is a daemon which automatically creates local print queues
when it discovers remote printers on the network.
For this it has to observe different sources for appropriate events:
- Bonjour: For IPP network printers and for remote CUPS queues of CUPS
1.6.x or newer
- Legacy CUPS broadcasts: For remote CUPS queues of CUPS 1.5.x or older
- D-Bus notifications: For load-balancing and maintenance of the local
queues.
In addition, timeouts are used to repeat failed attempts of creating and
taking down queues, to wait until all jobs are printed before
auto-shutdown, for expiring legacy CUPS queues, ...
All this is managed via a GLib GMainLoop and the list of discovered
printers is held in a global variable.
Now there come error reports from time to time, reporting crashes of
cups-browsed and as I did not find any problem with memory allocation
and freeing for the data structure I am assuming that the problem is
missing locking.
Probably (I am not sure how it works exactly) events like appearing or
disappearing printers, legacy CUPS broadcasts, ... make the main loop
starting threads to treat the events appropriately, as for example
adding and removing printers in the list of discovered printers.
Here probably race conditions come up when events happen close to each
other and the printer list gets manipulated by various threads at the
same time.
The solution would be to acquire a lock when starting to manipulate the
printer list and releasing the lock when done.
Now my qestion is, which functions I have to use for acquiring and
releasing locks when using GLib and GMainLoop? Probably it is not
correct to use the locks of pthread. Also it is probably best to use
Read/Write locks where only writing is exclusive but reading is allowed
to more than one thread at a time.
Any kind of help is welcome.
Till
More information about the ubuntu-devel
mailing list