Hello,

thank you for looking at my bug report.

On 05-Mar-09 21:14, John Hasler wrote:
> Chrony compiles with gcc-3.3.5 on i386 with this patch but most chrony
> commands hang (I didn't change libreadline).

Yes, my patch was incomplete. Sorry.

Please try the attached patch which has the missing amendment of
the memory allocation statement for 'ExtendedTable'.

Regards
Andreas Jochens

diff -urN ../tmp-orig/chrony-1.20/addrfilt.c ./addrfilt.c
--- ../tmp-orig/chrony-1.20/addrfilt.c  2002-03-01 00:27:08.000000000 +0100
+++ ./addrfilt.c        2005-03-10 08:33:35.599161694 +0100
@@ -45,7 +45,7 @@
 
 struct _TableNode;
 
-typedef struct _TableNode ExtendedTable[TABLE_SIZE];
+typedef struct _TableNode *ExtendedTable;
 
 typedef enum {DENY, ALLOW, AS_PARENT} State;
 
@@ -124,7 +124,7 @@
 
   if (node->extended == NULL) {
 
-    node->extended = MallocNew(ExtendedTable);
+    node->extended = MallocNew(sizeof(*ExtendedTable)*TABLE_SIZE);
 
     for (i=0; i<TABLE_SIZE; i++) {
       child_node = &((*(node->extended))[i]);
diff -urN ../tmp-orig/chrony-1.20/debian/control ./debian/control
--- ../tmp-orig/chrony-1.20/debian/control      2005-03-10 08:33:48.066266406 
+0100
+++ ./debian/control    2005-03-09 14:11:34.000000000 +0100
@@ -3,7 +3,7 @@
 Priority: extra
 Maintainer: John Hasler <[EMAIL PROTECTED]>
 Standards-Version: 3.5.8.0
-Build-Depends: debhelper, libreadline4-dev, texinfo, bison
+Build-Depends: debhelper, libreadline5-dev, texinfo, bison
 
 Package: chrony
 Architecture: any




-- 
To UNSUBSCRIBE, email to [EMAIL PROTECTED]
with a subject of "unsubscribe". Trouble? Contact [EMAIL PROTECTED]

Reply via email to