Bug#441967: cfengine2: Infinite loop with multiple class definition in editfiles

[Daniel Dehennin]

Package: cfengine2
Version: 2.1.22-1
Severity: important

Hello,

There is a bug in src/edittools.c which cause an infinite loop when defining
multiple classes in DefineClasses.

This is solved in svn, patch attached.

-- System Information:
Debian Release: lenny/sid
  APT prefers unstable
  APT policy: (500, 'unstable')
Architecture: amd64 (x86_64)

Kernel: Linux 2.6.22-1-amd64 (SMP w/2 CPU cores)
Locale: LANG=fr_FR.UTF-8, LC_CTYPE=fr_FR.UTF-8 (charmap=UTF-8)
Shell: /bin/sh linked to /bin/bash

Versions of packages cfengine2 depends on:
ii  debconf [debconf-2.0]      1.5.14        Debian configuration management sy
ii  debianutils                2.23.1        Miscellaneous utilities specific t
ii  libc6                      2.6.1-2       GNU C Library: Shared libraries
ii  libdb4.2                   4.2.52+dfsg-3 Berkeley v4.2 Database Libraries [
ii  libssl0.9.8                0.9.8e-8      SSL shared libraries
ii  perl                       5.8.8-7       Larry Wall's Practical Extraction 

cfengine2 recommends no packages.

-- debconf information:
  cfengine2/run_cfservd: true
  cfengine2/run_cfexecd: true
  cfengine2/run_cfenvd: false

=== modified file 'src/edittools.c'
--- src/edittools.c     2007-09-07 19:32:58 +0000
+++ src/edittools.c     2007-09-12 09:19:17 +0000
@@ -1640,7 +1640,7 @@
             
             sscanf(sp,"%[^,:.]",currentitem);
             
-            sp += strlen(currentitem) - 1;
+            sp += strlen(currentitem);
             
             AddClassToHeap(currentitem);
             }