diff options
| author | Volker Lendecke <vl@samba.org> | 2014-07-29 12:48:55 +0000 |
|---|---|---|
| committer | Michael Adam <obnox@samba.org> | 2014-07-31 18:49:46 +0200 |
| commit | dd4c019d9ffaa5453e536c8aaaa8e03463dc2959 (patch) | |
| tree | 0f2c87f98e58dadb03e653a16595e7ffc350e47f /lib | |
| parent | 6c9be497a3f41824411fe2507b4183d764a5f58e (diff) | |
| download | samba-dd4c019d9ffaa5453e536c8aaaa8e03463dc2959.tar.gz | |
debug: Fix debug_parse_params
The additional check this patch removes potentially drops some debuglevel
class settings. While not documented, it is possible to add a debug class
like for example "tdb:10" multiple times with multiple values. Later
settings overwrite earlier ones, but only if the overall number of
settings does not exceed the total number of available debug classes.
This patch changes our behaviour, but I can't imagine a situation where
someone relies on this check.
Signed-off-by: Volker Lendecke <vl@samba.org>
Reviewed-by: Michael Adam <obnox@samba.org>
Diffstat (limited to 'lib')
| -rw-r--r-- | lib/util/debug.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/lib/util/debug.c b/lib/util/debug.c index 73e5c5b485c..013f34c2c5f 100644 --- a/lib/util/debug.c +++ b/lib/util/debug.c @@ -392,7 +392,7 @@ static bool debug_parse_params(char **params) } /* Fill in new debug class levels */ - for (; i < debug_num_classes && params[i]; i++) { + for (; params[i]; i++) { bool ok; ok = debug_parse_param(params[i]); |