diff options
| author | Holger Hans Peter Freyther <zecke@selfish.org> | 2010-08-06 06:48:43 +0800 | 
|---|---|---|
| committer | Holger Hans Peter Freyther <zecke@selfish.org> | 2010-08-06 06:50:23 +0800 | 
| commit | 43558316bc4e6ddac92dd198b07a4803c21dc77b (patch) | |
| tree | abc157bc9668a4859e1b9f0c64caf507af16562b /src | |
| parent | c2b7f92a6c1ae3ad4f4865f3c2c69d4a3a5a279a (diff) | |
bsc_fd: Add a build option to be able to debug fd list corruption
bsc_register_fd on an already registered fd can corrupt the list
in a heavy way and make the select end in an infinite loop, this
code will workaround the issue and provide a way to set a breakpoint
at the right position in the code.
Diffstat (limited to 'src')
| -rw-r--r-- | src/select.c | 12 | 
1 files changed, 12 insertions, 0 deletions
diff --git a/src/select.c b/src/select.c index 2f6afa7f..f52b0a0c 100644 --- a/src/select.c +++ b/src/select.c @@ -19,6 +19,8 @@   */  #include <fcntl.h> +#include <stdio.h> +  #include <osmocore/select.h>  #include <osmocore/linuxlist.h>  #include <osmocore/timer.h> @@ -48,6 +50,16 @@ int bsc_register_fd(struct bsc_fd *fd)  	if (fd->fd > maxfd)  		maxfd = fd->fd; +#ifdef BSC_FD_CHECK +	struct bsc_fd *entry; +	llist_for_each_entry(entry, &bsc_fds, list) { +		if (entry == fd) { +			fprintf(stderr, "Adding a bsc_fd that is already in the list.\n"); +			return 0; +		} +	} +#endif +  	llist_add_tail(&fd->list, &bsc_fds);  	return 0;  | 
