snowcrash+sqlite <[EMAIL PROTECTED]> wrote:
> > ...
> > On my system (which is generic as far as I know) this runs
> > to completion without any errors.
> > ...
> > I'm not sure what the difference is.
>
> tbh, i have _no_ concrete idea either as to why you are NOT seeing
> these issues on your MacBook :-/
FWIW, my MacBook has nothing special installed. The "About This Mac"
dialog box says "Mac OS X Version 10.4.8. 2GHz Intel Core Duo". I
searched through the Makefile that configure generated and there was
no sign of a -static flag anywhere. "gcc -v" says version 4.0.1,
(Apple Computer, Inc. build 5250). I'm really curious to know why
it works and nobody elses does.
> diff -ru sqlite-3.3.8ORIG/ sqlite-3.3.8
> diff -re sqlite-3.3.8ORIG/Makefile.in sqlite-3.3.8/Makefile.in
> --- sqlite-3.3.8ORIG/Makefile.in 2007-01-02 13:17:56.000000000 -0800
> +++ sqlite-3.3.8/Makefile.in 2007-01-02 13:39:48.000000000 -0800
> @@ -271,7 +271,7 @@
> # of the most recently modified source code file
> #
> last_change: $(SRC)
> - cat $(SRC) | grep '$$Id: ' | sort -k 5 | tail -1 \
> + cat $(SRC) | grep '$$Id: ' | sort -k 5 | tail -n 1 \
> | $(NAWK) '{print $$5,$$6}' >last_change
yeah, yeah. I used to be able to say "sort -5" too. I resisted
making the change to "-k 5" for as long as I could. Changes to
the unix interface like this come from the crowd who things that
it is more important to be consistent than correct. In reality,
consistency is the hobgloblin of little minds. I've been happily
typing "tail -1" since before many of the readers of this message
were born, and I will continue to resist the change to "tail -n 1"
for as long as I can. Call me cranky if you want. That is the
privilege of people who have seen as much water under the bridge
as I have.
> diff -ru sqlite-3.3.8ORIG/tool/lemon.c sqlite-3.3.8/tool/lemon.c
> --- sqlite-3.3.8ORIG/tool/lemon.c 2007-01-02 13:17:55.000000000 -0800
> +++ sqlite-3.3.8/tool/lemon.c 2007-01-02 13:30:36.000000000 -0800
> @@ -1322,15 +1322,15 @@
> va_start(ap, format);
> /* Prepare a prefix to be prepended to every output line */
> if( lineno>0 ){
> - sprintf(prefix,"%.*s:%d: ",PREFIXLIMIT-10,filename,lineno);
> + snprintf(prefix,sizeof prefix,"%.*s:%d:
> ",PREFIXLIMIT-10,filename,lineno);
Lemon was written before snprintf() was invented. And it is a
developer tool, not really subject to exploitation. Note that none
of this code is actually in the SQLite core. Note also the
careful limiting of the resulting string length using ".*"
before the "s" in "%s". This is not an important patch and does
not close any real bug that I can see. It probably just makes
some gee-whiz syntax checker tool stop complaining. Whatever...
--
D. Richard Hipp <[EMAIL PROTECTED]>
-----------------------------------------------------------------------------
To unsubscribe, send email to [EMAIL PROTECTED]
-----------------------------------------------------------------------------
