On Wed, 2009-11-18 at 11:34 -0800, Zach Welch wrote:
> On Wed, 2009-11-18 at 11:23 -0700, David Brownell wrote:
> > On Wednesday 18 November 2009, Zach Welch wrote:
> > > On Tue, 2009-11-17 at 09:39 -0800, Zachary T Welch wrote:
> > > > Hi all,
> > > > 
> > > > This series modularizes the startup.tcl file, putting various pieces
> > > > of the system where they belong.  Instead of linking it into the 
> > > > library,
> > > > provide it as a parameter to the command_init() routine from openocd.c.
> > > > Fixes the layering violations that this built-in TCL code created.
> > > 
> > > Pushed, with a fix in the second patch to add the new files to be
> > > distributed by the automake rules.
> > 
> > Now OpenOCD reliably coredumps on startup...
> 
> Uh-oh  I though that I had tested those changes fairly well, but I
> apparently made a mistake.  I'll try to figure out what's going on and
> commit a fix shortly.

Okay, this seems like it fixed the crash for me:

diff --git a/src/openocd.h b/src/openocd.h
index 70e3ee0..a91d46f 100644
--- a/src/openocd.h
+++ b/src/openocd.h
@@ -37,6 +37,6 @@ void openocd_sleep_prelude(void);
 void openocd_sleep_postlude(void);

 /// provides a hard-coded command environment setup
-extern const char *openocd_startup_tcl;
+extern const char openocd_startup_tcl[];

 #endif

If that works for you, then I'll push it.  Why it works for me, I have
no clear idea... any insight would be welcome.

Cheers,

Zach
_______________________________________________
Openocd-development mailing list
[email protected]
https://lists.berlios.de/mailman/listinfo/openocd-development

Reply via email to