Make sure that the args array is initialized
CID:394
Signed-off-by: Nelson Ferreira <nelson.ferreira@ieee.org>

Make sure dl.top_clip is initialized
CID:406
Signed-off-by: Nelson Ferreira <nelson.ferreira@ieee.org>

Avoid doing decode_buffer twice when not doing clash
detection
CID:430
Signed-off-by: Nelson Ferreira <nelson.ferreira@ieee.org>

Remove unneeded initialization of command_builder pointer.
CID:431
Signed-off-by: Nelson Ferreira <nelson.ferreira@ieee.org>

Do not assign the return of setlocale(LC_ALL,"C") to locale var
since it will be no longer used afterwards.
CID:432
Signed-off-by: Nelson Ferreira <nelson.ferreira@ieee.org>

Remove unused value of eckey
CID:435
Signed-off-by: Nelson Ferreira <nelson.ferreira@ieee.org>

Make sure that check and change of state is done on the
mutual exclusion zone. move the sleep outside of it.
CID:440

Signed-off-by: Nelson Ferreira <nelson.ferreira@ieee.org>

make sure the optable_index cannot be negative, instead
an assertion is made.
CID:117,116,115,114,111,110,109,88,87
Signed-off-by: Nelson Ferreira <nelson.ferreira@ieee.org>

Make sure avfc is not NULL before dereferencing
CID:457
Signed-off-by: Nelson Ferreira <nelson.ferreira@ieee.org>

Make sure that avfc is non null before de-reference.
CID:458
Signed-off-by: Nelson Ferreira <nelson.ferreira@ieee.org>

Make sure the pointers are valid before dereferencing
CID:461
Signed-off-by: Nelson Ferreira <nelson.ferreira@ieee.org>

Use strncat instead of strcat, even in a paranoid fashion.
CID:463
Signed-off-by: Nelson Ferreira <nelson.ferreira@ieee.org>

Make sure that mtp has the proper play state before entering
the loop.
CID:466
Signed-off-by: Nelson Ferreira <nelson.ferreira@ieee.org>

Fix stupid typo on previous fix, that broke build.
Signed-off-by: Nelson Ferreira <nelson.ferreira@ieee.org>

Fix several FILE* leaks in ssl
CID:193,194,195,196,197,198
Signed-off-by: Nelson Ferreira <nelson.ferreira@ieee.org>

Make implementation comment about issues with strlen of
NULL pointers match the implementation
Signed-off-by: Nelson Ferreira <nelson.ferreira@ieee.org>

Fix several invalid uses of negative returns on acessing an array
due to ase_optable_index
CID:87,88,109,110,111,114,115,116,117
Signed-off-by: Nelson Ferreira <nelson.ferreira@ieee.org>

Allocate and copy ures when it would be the return address.
CID:200
Signed-off-by: Nelson Ferreira <nelson.ferreira@ieee.org>

Move variable path definition to avoid out-of-scope references
to the memory location.
CID:201
Signed-off-by: Nelson Ferreira <nelson.ferreira@ieee.org>

CID:326,325
Use snprintf instead of sprintf. Although current sizes and formatting
string are fine (since it is only integer printing), it won't hurt to
limit the buffer.
Signed-off-by: Nelson Ferreira <nelson.ferreira@ieee.org>

Use snprint instead of sprintf. One of the uses of
this CID:342 was of valid concern.
Signed-off-by: Nelson Ferreira <nelson.ferreira@ieee.org>

Use snprint even if there is no issue with given size...
Actually reduced it, so that the stack is less likely to
blow up.
CID:343
Signed-off-by: Nelson Ferreira <nelson.ferreira@ieee.org>

Get rid of harmless warning about ignoring result of getc, while ignoring ESC ISO2022 sequences.
CID:11
Signed-off-by: Nelson Ferreira <nelson.ferreira@ieee.org>

Avoid potential race condition and check only mtp as loop
exit condition, which is updated at each switch condition
that may change the state and also updated under lock.

Signed-off-by: Nelson Ferreira <nelson.ferreira@ieee.org>

Signed-off-by: Nelson Ferreira <nelson.ferreira@ieee.org>
Correct silly mistake in previous locking fix. Use UNLOCK instead
of LOCK.

CID:454

Avoid resource leaks in redisplay (runes mostly).
CID:175,176,177,186,187,188,189,190,191
Signed-off-by: Nelson Ferreira <nelson.ferreira@ieee.org>

Enclosed the media stop status assignment inside the
mutex zone. Also made sure any "derived" vars were updated
CID:440,441,442
Signed-off-by: Nelson Ferreira <nelson.ferreira@ieee.org>

Use ssize_t for string length and return from read calls
CID:130,131,132,133
Signed-off-by: Nelson Ferreira <nelson.ferreira@ieee.org>

Initialize the return object to Qnil
CID:452
Signed-off-by: Nelson Ferreira <nelson.ferreira@ieee.org>

Check for NULL return from getenv on pdump_load. These
are made with assert since it is so early there is no
recovery possible or needed.
CID:443

Signed-off-by: Nelson Ferreira <nelson.ferreira@ieee.org>

Check for NULL return from getenv
CID:444
Signed-off-by: Nelson Ferreira <nelson.ferreira@ieee.org>

Remove superfluous pointer check which seems to trigger
CID:445
Signed-off-by: Nelson Ferreira <nelson.ferreira@ieee.org>

Merge branch 'merges'

Fix potential usage of negative file handle CID:119

Fix potential usage of negative file descriptor
CID:120

Fix potential usage of negative file handle
CID:119
Signed-off-by: Nelson Ferreira <nelson.ferreira@ieee.org>

Make loop explicit in signal_if_x_error CID:61
Signed-off-by: Nelson Ferreira <nelson.ferreira@ieee.org>

Shutup unchecked call return value (CID:13)
Signed-off-by: Nelson Ferreira <nelson.ferreira@ieee.org>

Fix  potential array overrun of static due to sign bit expansion
CID:141

Fix undefined order of eval in while condition.
CID:30

Fix potential stack/string overflow warning
CID:379

Fix potential stack overflow (via string overflow)
CID:380

Use snprintf instead of sprint
CID:344 SECURE_CODING

Fix strcpy from environment variable into fixed size buffer.  Fix strcat from argument into fixed size buffer (call to expand_filename).
CID:381

Fix unchecked strcpy from commandline argument into fixed size buffer
CID:382

Merge branch 'merges'

Fix the ent-mpc issues of Bug 126.

Check the result of sxemacs_stat in Fwrite_region_internal (CID:14)

Merge branch 'merges'

Prevent harmless COVERITY warning message of the BAD_ALLOC_STRLEN checker on relative_filename about the usage of pointer+1 inside strlen as parameter to xnew.

Changed it because it is indeed more maintainable to walk past the '/' and use that afterwards.

Fix CID:436 Unused value

Fix CID:437 Unused value

Merge http://git.sxemacs.org/sxemacs

Merge branch 'tooltalkless'

Purge SXEmacs of tooltalk -- remainding tidy up

Finger's crossed, this is the END of tooltalk.  Yay!

* PROBLEMS: Delete the tooltalk stuff.

* etc/tests/external-widget/Makefile (CFLAGS): Remove the
-DTOOLTALK flag

* etc/tests/external-widget/test-ew-motif.c (handle_tt_input):
Remove.  It was tooltalk.

* etc/tests/external-widget/test-ew-motif.c
(HxInitializeToolTalk): Ditto.

* etc/tests/external-widget/test-ew-motif.c (main): Don't call
HxInitializeToolTalk().

* info/external-widget.texi (Using an External Client Widget):
Don't mention tooltalk.

* src/events/event-stream.c (execute_internal_event): Mention in
comments relating to tooltalk to revisit with D-Bus

* src/ui/X11/ExternalClient.h (XtNuseToolTalk): Remove.

* src/ui/X11/ExternalClient.h (XtCUseToolTalk): Ditto.

* .gitignore: Remove tooltalk directory.

Signed-off-by: Steve Youngs <steve@sxemacs.org>

Purge SXEmacs of tooltalk cruft -- src, lisp

This kills off pretty much all of the tooltalk.  But I expect a couple
more (at least) changesets to completely tidy it up.

* lisp/Makefile.am (corelispels): Remove check-features.el.  It
  was just tooltalk and sparcwork cruft.

* lisp/dumped-lisp.el (packages-hardcoded-lisp): Remove tooltalk
  related stuff.  Was all just comments anyway.

* lisp/subr.el (error): Remove tooltalk-error

* src/ui/Makefile.am (libsxeui_a_headers): Remove tooltalk.h

* src/ui/X11/ExternalClientP.h: Remove the use_tooltalk member
  from the ExternalClientPart struct.

* src/ui/X11/ExternalClient.c: Remove tooltalk cruft.

* src/ui/X11/ExternalClient.c (send_tooltalk_handshake): Remove.

* src/ui/X11/ExternalClient.c (tt_callback): Ditto.

* src/symsinit.h: Remove tooltalk cruft.

* src/procimpl.h: Remove tooltalk from process_methods

* src/process.c (connected_via_filedesc_p): return 0 regardless.
  It was only ever non-0 with a tooltalk connection.  'nuff said.

* src/process-unix.c (unix_tooltalk_connection_p): Remove.

* src/process-unix.c (process_type_create_unix): Remove tooltalk cruft.

* src/lrecord.h: Remove tooltalk cruft.

* src/inline.c: Ditto.

* src/gdbinit: Ditto.

* src/emacs.c (shut_down_emacs): Ditto.

* src/dbxrc (Usage): Ditto.

Signed-off-by: Steve Youngs <steve@sxemacs.org>

Purge SXEmacs of tooltalk madness -- test suite

This eradicates tooltalk from our test suite.

* tests/Makefile.am (tooltalk_tests): Removed.

* tests/Makefile.am (EXTRA_DIST): Remove tooltalk_tests.

Signed-off-by: Steve Youngs <steve@sxemacs.org>

Purge SXEmacs of tooltalk and CDE cruft -- docs, autoconf

Tooltalk was some old Sun/Sparcworks thingamajiggy.  Nobody uses it
anymore and it has been superseded by D-Bus anyway.  Even though SXEmacs
doesn't yet have full (any?) D-Bus support we're nuking this silly
tooltalk madness now.

This changeset removes the tooltalk cruft from our build chain autotool
stuff and from the docs.

* configure.ac: Remove all the tooltalk and CDE cruft.

* m4/sxe-summary.m4 (SXE_SUMMARY): Ditto.

* etc/PACKAGES: Ditto.

* info/sxemacs-faq.texi (Q4.4.2): Removed.
  Plus nuke other mentions of tooltalk.

* info/sxemacs/packages.texi (Available Packages): Remove
  tooltalk. Yes I know that the tooltalk XEmacs package is still
  available, but there's not much point in saying so in the SXEmacs
  manual if SXEmacs doesn't support tooltalk at all.

* info/lispref/objects.texi: Remove all the tooltalk cruft.

* info/lispref/lispref.texi: Ditto.

* info/lispref/hooks.texi (Standard Hooks): Ditto.

* info/lispref/errors.texi (Standard Errors): Ditto.

* info/internals/internals.texi: Ditto.

* info/lispref/ldap.texi (LDAP Support): Set prev node to
  X-Windows.  Skips tooltalk.

* info/lispref/x-windows.texi (X-Windows): Set next node to LDAP
  Support.  Skips tooltalk.

Signed-off-by: Steve Youngs <steve@sxemacs.org>

Merge branch 'merges'

- [FIX] ffi-curl, avoid signalling

Signed-off-by: Zajcev Evgeny <zevlg@yandex.ru>

ffi-wand, removed some dups

Signed-off-by: Zajcev Evgeny <zevlg@yandex.ru>

Merge branch 'merges'

- revert, avoid raw format check

Signed-off-by: Zajcev Evgeny <zevlg@yandex.ru>

- [add] wand: partial support for GM lib
- [fix] safe version of ffi-magic

misc enhs

Merge branch 'merges'

Sundry FFI fixes (see bug #127)

Here are a couple of FFI fixes that bring us closer to fixing bug #127

* lisp/ffi/ffi-sqlite.el (sqlite-bind-value): Use sqlite-STATIC
instead of sqlite-TRANSIENT

* src/effi.c (ffi_call_using_libffi): Temporarily turn of async
timeouts when calling ffi_call()

* src/signal.c (stop_async_timeouts): Make it available
externally.

* src/signal.c (start_async_timeouts): Ditto.

Signed-off-by: Steve Youngs <steve@sxemacs.org>

- [enh] ffi-magic with safe strings

Fix building with current git FFmpeg

* m4/sxe-mm.m4 (SXE_MM_CHECK_FFMPEG): Turn off -Werror in the test
to see if url_fopen() needs ByteIOContext**.

That stuff is now deprecated in current FFmpeg git repo.  This
patch allows SXEmacs to build with it, although, with loads of
warnings.  Yeah, I'm on it. :-)

Signed-off-by: Steve Youngs <steve@sxemacs.org>

Merge http://git.sxemacs.org/sxemacs

Update and fix our .desktop file.

* etc/sxemacs.desktop.in (StartupNotify): Set to false.  I'm
almost certain we don't do this.

* etc/sxemacs.desktop.in (MimeType): Update to include all the
MIME types I can think of that we support.

* etc/sxemacs.desktop.in (Categories): The spec says that if we're
a "TextEditor", we should also be a "Utility".  So add that.

Signed-off-by: Steve Youngs <steve@sxemacs.org>

Libpng 1.5 fix.

One of the things they did in libpng 1.5 was to remove all the stuff in
1.4 and earlier that had been marked deprecated.  Of course we were using
some of those deprecated things.  This patch fixes that and brings back
PNGs to SXEmacsen built with the latest libpng.

* src/ui/glyphs-eimage.c (png_instantiate): Rewrite and update for
libpng 1.5.x.  Also, handle interlaced PNGs.

Signed-off-by: Steve Youngs <steve@sxemacs.org>

Merge http://git.sxemacs.org/sxemacs

Merge branch 'merges'

Merge branch 'merges', remote-tracking branch 'njsf/master' into merges

* src/mule/input-method-xlib.c: Fix a header due to
        EmacsFrame.h move that had been overlooked.

Signed-off-by: Nelson Ferreira <nelson.ferreira@ieee.org>

Even friendlier advice to the novice git SXEmacs
contributor.

* autogen.sh: also state one needs to git push --tag
* autogen.sh: if the detected TREE_VERSION differs from the expected, issue a warning

Signed-off-by: Nelson Ferreira <nelson.ferreira@ieee.org>

* src/ui/glyphs.c: properly allow Q_mask_{file,data},
                   Q_hotspot_{x,y}, Q_foreground,
                   Qbackground on tty only builds.

* src/ui/glyphs.c(syms_of_glyphs): ditto.

* src/ui/glyphs.c(xface_normalize): properly handle
                   tty only builds by returning nil
                   when HAVE_WINDOW_SYSTEM is not
                   defined and conversion is needed.

Signed-off-by: Nelson Ferreira <nelson.ferreira@ieee.org>

Build improvements

* autogen.sh: when outside of "proper" git repos
              try to be friendlier by using the
              defaults and suggest the tag command.

* src/Emacs{Frame,Manager,Shell}*, src/External*:
              moved into src/ui/X11 as that is the
              proper place for those files

* src/Makefile.am, src/ui/X11/Makefile.am:
              adjust targets for move of the files

* src/mule/input-method-motif.c: adjust header path
* src/ui/lwlib/lwlib-Xlw.c: ditto

Signed-off-by: Nelson Ferreira <nelson.ferreira@ieee.org>

Couple more warning fixes

* src/media/sound-jack.c (sound_jack_shutdown_cbfun): Use %p mask
to prevent compiler warning.

* src/media/media.c (__add_prop): Drop the artificial attribute to
prevent a compiler warning.

Signed-off-by: Steve Youngs <steve@sxemacs.org>

Fix a few more compiler warnings.

* src/skiplist.h: #if 0 out some as yet unused function
declarations.  Prevents compiler warnings.

* src/openssl.c (Fossl_ssl_cipher_description):
SSL_get_current_cipher returns a const, so needs to be stored in a
const.  Prevents compiler warning.

* src/openssl.c (Fossl_ssl_cipher_version): Ditto.

* src/openssl.c (Fossl_ssl_cipher_name): Ditto.

* src/openssl.c (Fossl_ssl_cipher_bits): Ditto.

Signed-off-by: Steve Youngs <steve@sxemacs.org>

Fixes all byte-compiler warnings in our elisp.

* lisp/ffi/ffi-wand.el
(Wand-mode-insert-info, Wand-mode-select-region): Use
#'when-boundp to avoid byte-compiler warning.  Also require
wid-edit for same.

* lisp/help.el: Globally declare #'find-function #'find-variable
and #'view-scroll-lines-up at compile time to avoid byte-compiler
warnings.

* lisp/minibuf.el (x-read-color-completion-table): defvar it at
compile time to avoid byte-compiler warning.

Signed-off-by: Steve Youngs <steve@sxemacs.org>

Fix several compiler warnings in C sources.

This takes care of most of the compiler warnings that have crept back into
the C sources.  There are a couple more left, but they will require us to
catch up with current versions of libpng and ffmpeg, so not quite as
trivial as the rest of this patch.

* modules/ase/ase-metric.c (ase_metric_prnt): Don't try to cast
XASE_METRIC_DIST to unsigned int, use %p printf mask instead.
Prevents compiler warning about casting pointer to int of
different size.

* src/ui/TTY/terminfo.c (emacs_tparam): cast string to char* to
keep tparm/ncurses happy.
Prevents compiler warning.

* src/ui/X11/glyphs-x.c (x_subwindow_query_geometry): Cast dx, dy
to int.
Prevents a compiler warning about wrong types.

* src/media/sound-alsa.h (sound_alsa_aj_data_s): samplerate is an
unsigned int.
Prevents a compiler warning.

Signed-off-by: Steve Youngs <steve@sxemacs.org>

Add Last version ChangeLog and tweak SPPM

* info/sppm.texi (Making Releases): Tweak the instructions in a
couple of places.

Signed-off-by: Steve Youngs <steve@sxemacs.org>

SXEmacs 22.1.14 (Geo) is Released!

* INSTALL: Couple minor arch -> git updates.

* PROBLEMS: Update version it applies to

* etc/NEWS: Release day update

* autogen.sh (sxemacs_codename): Geo
  (TREE_VERSION): Bump

* info/sxemacs/sxemacs.texi (Top): Bump version the manual
corresponds to.

* info/sppm.texi: Update version and date macros

Signed-off-by: Steve Youngs <steve@sxemacs.org>

Merge branch 'merges'

Merge branch 'sppm-git'

Refactored a bit the SPPM portion of Patches and Version control. Now Patches only deals with how to send a patch, and Version Control shows how to setup repos. Cross references were added.

Signed-off-by: Nelson Ferreira <nelson.ferreira@ieee.org>

Merge branch 'merges'

Merge branch 'sppm-update'

A note on PRs and associated git hooks in sppm.texi 13.0.1

Signed-off-by: Horst Günther Burkhardt III <horst@sxemacs.org>

Quick typo fix in SPPM

Signed-off-by: Steve Youngs <steve@sxemacs.org>

Merge branch 'sppmupd'

Update SPPM converting all the tla stuff to git stuff

This gives the SPPM the once over to cover all the stuff that changes
because we now use git.

* info/sppm.texi (Patches): Rewrite for git
  (Publicly Accessible Repo): New node.
  (Private Repo): New node.
  (Making Releases): Rewrite for git.
  (Version Control): Rewrite for git.

Signed-off-by: Steve Youngs <steve@sxemacs.org>

Update the .gitignore to cater for in-tree builds

* .gitignore: Update to cater for in-tree builds

Signed-off-by: Steve Youngs <steve@sxemacs.org>

Convert the tla-generated versioning info to git-generated

This changeset converts all of our sexy automagic versioning info
from being tla-based to being git-based.  The old tla variables
are now gone, replaced with git equivalents...

    SXEMACS_ARCH_VERSION -> SXEMACS_GIT_VERSION
    sxemacs_arch_version -> sxemacs_git_version
    sxemacs-arch-version -> sxemacs-git-version
    SXEMACS_MAIN_ARCH_VERSION -> removed.

Also, a .gitignore has been added to the repo.

* src/emacs.c (vars_of_emacs): New variable, Vsxemacs_git_version,
sxemacs-git-version

* m4/sxe-summary.m4: tla -> git

* lisp/version.el (emacs-version): Use sxemacs-git-version instead
of the old arch one.

* lisp/obsolete.el (sxemacs-arch-version): Obsolete it.

* lisp/build-rpt.el (build-rpt-subject): Use SXEMACS_GIT_VERSION
instead of the old arch one.

* configure.ac: tla -> git

* autogen.sh: Convert all the tla/arch stuff to git stuff.

Signed-off-by: Steve Youngs <steve@sxemacs.org>

Initial git import