Coverity updates/fixes from Nelson

* merges: (21 commits)
  Coverity: Missing break: CID 67
  Coverity: Missing break: CID 68
  Coverity: Missing break: CID 69
  Coverity: Missing break: CID 72
  Coverity: Missing break: CID 73
  Coverity: Missing Break: CID 76
  Coverity: Missing lock: CID 81
  Coverity: Missing Lock: CID 82
  Fix build
  Coverity: Negative returns: CID 91
  Coverity: Negative RETURNS: CID 106
  Coverity: Negative returns: CID 107
  Coverity: Negative Returns: CID 121
  Coverity: Reverse INULL: CID 202
  Coverity: Reverse INULL: CID 130
  Coverity: Reverse Negative: CID 210
  Coverity: UNINIT: CID 395
  Coverity: Uninit: CID 396
  Coverity: UNINIT: CID 602
  Coverity: Uninit: CID 603
  ...

Coverity: Missing break: CID 67

* src/chartab.c (char_table_type_to_symbol): Add break to account
for fatal crash scenario.

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

Coverity: Missing break: CID 68

* src/events/events.c (Fevent_properties): Add break to account
for fatal crash scenario.

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

Coverity: Missing break: CID 69

* src/events/events.c (event_equal): Add break to account for
fatal crash scenario.

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

Coverity: Missing break: CID 72

* src/mule/file-coding.c (Fcoding_system_type): Add break to
account with fatal crash situation.

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

Coverity: Missing break: CID 73

* src/mule/file-coding.c (eol_type_to_symbol): Add break to
account with fatal crash situation.

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

Coverity: Missing Break: CID 76

* src/ui/glyphs.c (Fglyph_type): Add break to account for fatal
crash situation.

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

Coverity: Missing lock: CID 81

* src/semaphore.h (sxe_refcounter_finish): Be "correct" and
pedantic about locking in refcounter_init

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

Coverity: Missing Lock: CID 82

* src/semaphore.h (sxe_refcounter_new): Fix the refcounter finish
locking

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

Fix build

* src/ui/X11/xgccache.c (gc_cache_lookup): Fix build (missing } )

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

Coverity: Negative returns: CID 91

* lib-src/gnuserv.c (handle_unix_request): Handle error in
accept...

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

Coverity: Negative RETURNS: CID 106

* src/indent.c (vpix_motion): Try to handle more safely the
crashing fatally scenario...

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

Coverity: Negative returns: CID 107

* src/dumper.c (pdump): Make sure not to take invalid paths even
when abort is a no-op...

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

Coverity: Negative Returns: CID 121

* src/ui/TTY/objects-tty.c (tty_font_spec_matches_charset): Make
sure to abort in case of invariant violation.

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

Coverity: Reverse INULL: CID 202

* src/mem/ralloc.c (update_heap_bloc_correspondence): Be extra
safe and do "abort" on violation of stated (in comment) invariant.

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

Coverity: Reverse INULL: CID 130

* src/media/sound.c (audio_device_finalise): Do nothing if header is NULL

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

Coverity: Reverse Negative: CID 210

* src/process-unix.c (unix_kill_child_process): Make sure to test
for subtty non negative before calling process_signal_char.

* src/process-unix.c (process_signal_char): Make sure fd is non
negative.

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

Coverity: UNINIT: CID 395

* src/extents.c (extent_fragment_update): Make sure the glyph_bloc
is initialized.

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

Coverity: Uninit: CID 396

* src/fileio.c (file-truename) Make sure path is initialized before use.

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

Coverity: UNINIT: CID 602

* src/ui/X11/xgccache.c: More UNINIT cleanup

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

Coverity: Uninit: CID 603

* src/ui/lwlib/lwlib-Xaw.c (xaw_generic_callback): Make sure
user_data is either initialized or not used :)

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

Fix get_dyna_doc CHAR_IO issue sync

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

Coding style cleanups from Aidan

* from-aidan:
  Improve conformance with usual coding style in effi.c.

Improve conformance with usual coding style in effi.c.

Qabcdef should hold the perfectly normal, interned symbol abcdef, its
C variable should not be named Q_abcdef, that would imply the keyword
:abcdef.

        * effi.c:
        Only declare the symbols corresponding to foreign types if they're
        not declared elsewhere. Follow the usual convention as above.
        * effi.c (FFI_TPTR): Change symbol names here.
        * effi.c (Fffi_basic_type_p): Ditto.
        * effi.c (ffi_canonicalise_type): Ditto.
        * effi.c (Fffi_size_of_type): Ditto.
        * effi.c (Fmake_ffi_object): Ditto.
        * effi.c (Fffi_make_pointer): Ditto.
        * effi.c (Fffi_load_library): Ditto.
        * effi.c (ffi_type_align): Ditto.
        * effi.c (ffi_fetch_foreign): Ditto.
        * effi.c (Fffi_fetch): Ditto.
        * effi.c (Fffi_aref): Ditto.
        * effi.c (Fffi_store): Ditto.
        * effi.c (Fffi_aset): Ditto.
        * effi.c (Fffi_address_of): Ditto.
        * effi.c (Fffi_lisp_object_to_pointer): Ditto.
        * effi.c (extffi_setup_argument): Ditto.
        * effi.c (Fx_device_display): Ditto.
        * effi.c (ffi_callback_call_x86): Ditto.
        * effi.c (Fffi_make_callback): Ditto.
        * effi.c (syms_of_ffi): Use DEFSYMBOL where appropriate, document an
        oddity with Qffiobjectp.
        * general-slots.h: Move Qshort, Qvoid here.
        * lisp.h: Remove Qshort, Qvoid from here.
        * map.c: Ditto.
        * map.c (map_LTX_init): Ditto.
        * mule/file-coding.c: Ditto.
        * mule/file-coding.c (syms_of_file_coding): Ditto.
        * mule/file-coding.h: Ditto.

Signed-off-by: Aidan Kehoe <kehoea@parhasard.net>
Signed-off-by: Steve Youngs <steve@sxemacs.org>

Coverity fixes from Nelson

* merges:
  Coverity: FORWARD NULL: CID 54, 53, 52
  Coverity: Forward NULL: CID 59, 58, 56
  Coverity: CHAR_IO: CID 5,6,7,9
  Coverity: UNINIT: CID 393
  Coverity: UNINIT: CID 397
  Coverity: UNINIT: CID 398
  Coverity: UNINIT: CID 399
  Coverity: Uninit: CID 400
  Coverity: UNINIT: CID 401
  Fix build
  Fix build
  Coverity: UNINIT: CID 402
  Coverity: UNINIT: CID  403
  Coverity: UNINIT: CID 405, 404
  Coverity: UNUSED: CID 427
  Coverity: Forward NULL: CID 439
  Coverity: UNINIT: CID 599
  Coverity: UNINIT: CID 600
  Coverity: UNINIT: CID 601

Coverity: FORWARD NULL: CID 54, 53, 52

* src/ui/X11/xgccache.c (gc_cache_lookup): Refactor abort() flow
through to shut up coverity.

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

Coverity: Forward NULL: CID 59, 58, 56

* src/ui/lwlib/lwlib-Xaw.c (xaw_generic_callback): Make sure you
do not dereference possible NULL val.

* src/ui/lwlib/lwlib-Xaw.c (wm_delete_window): Make sure widget
and kids are only used when valid.

* src/ui/lwlib/lwlib.c (replace_widget_value_tree): Make sure node
is only used if != NULL.

* src/ui/lwlib/lwlib.c (lw_map_widget_values): Make sure info is
only used if != NULL.

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

Coverity: CHAR_IO: CID 5,6,7,9

* lib-src/make-docfile.c (skip_white): Use proper return type of getc
* lib-src/make-docfile.c (read_lisp_symbol): Ditto
* src/signal.c (interrupt_signal): Ditto
* lib-src/make-docfile.c (scan_c_file): Do proper casting

* lib-src/make-docfile.c (get_dyna_doc): Let the casting
implementation do the truncation.

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

Merge branch 'njsf-cov' into for-steve

Coverity: UNINIT: CID 393

* src/print.c (write_string_to_stdio_stream): handle the case
where the transcoding fails.

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

Coverity: UNINIT: CID 397

* src/buffer.c (directory_is_current_directory): Handle the case
of failed transcoding by attempting to use the non-transcoded
version. There is a high chance it is proper. Worst case it will
also fail, like the bad conversion would have prompted.

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

Coverity: UNINIT: CID 398

* src/effi.c (Fffi_store): Make sure tmp is only used in the mule
case if transcoding was proper.

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

Coverity: UNINIT: CID 399

* src/effi.c (Fffi_store): Make sure val_ext is only used if
transcoding was successful.

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

Coverity: Uninit: CID 400

* src/emacs.c (decode_path): make sure to return Qnil if
transcoding failed.

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

Coverity: UNINIT: CID 401

* src/media/media.c (Fmake_media_stream): make sure the stream is
only created if transcoding was ok.

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

Merge branch 'njsf-cov' into for-steve

Fix build

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

Fix build

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

Coverity: UNINIT: CID 402

* src/print.c (Falternate_debugging_output): Make sure to handle
the case where transcoding failed.

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

Coverity: UNINIT: CID  403

* src/ui/X11/gui-x.c (button_item_to_widget_value): Make sure
intname is only used if properly transcoded.

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

Coverity: UNINIT: CID 405, 404

* src/ui/X11/frame-x.c (x_set_frame_properties): Make sure extval
is only used if properly transcoded.

* src/ui/X11/select-x.c (lisp_data_to_selection_data): Ditto.

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

Coverity: UNUSED: CID 427

* src/ui/insdel.c (init_buffer_text): Get rid of unused

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

Coverity: Forward NULL: CID 439

* src/fns.c (concat): Manually skip and abort when
string_result_ptr is NULL. Coverity is actually right
here, because when we are in a fatal error handling,
assert will be a no-op, so we better just put an
abort for the other cases and not try to dereference...

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

Coverity: UNINIT: CID 599

* src/emacs.c (run_emacs_from_temacs): Make sure the arguments
were really transcoded before using them

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

Coverity: UNINIT: CID 600

* src/ui/X11/objects-x.c (x_find_charset_font): Make sure intname
is only used if transcoding worked properly.

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

Coverity: UNINIT: CID 601

* src/ui/glyphs-eimage.c (jpeg_instantiate): Make sure bytes is
only used if properly transcoded.

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

Coverity fixes from Nelson

* merges:
  Add a couple of missing semicolons on last commit
  Fix build
  Add some sanity assertions to format conversions.
  Coverity: UNINIT: CID 409,408,407,406
  Coverity: UNINIT: CID 410
  Coverity: UNREACHABLE: CID 426, 425, 424, 423, 422, 421
  Coverity: UNINIT: CID 453
  Coverity: Uninit: CID 473
  Coverity: Char IO: CID 595
  Coverity: No effect: CID 598
  Coverity: LOCK: CID 597

Add a couple of missing semicolons on last commit

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

Fix build

* src/buffer.h (TO_EXTERNAL_FORMAT): Move assertion up so that
macro can be used as single statement.

* src/buffer.h (TO_INTERNAL_FORMAT): Ditto.

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

Add some sanity assertions to format conversions.

* src/buffer.h (TO_EXTERNAL_FORMAT): Add additional assert of
validity after conversion.

* src/buffer.h (TO_INTERNAL_FORMAT): Ditto.

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

Coverity: UNINIT: CID 409,408,407,406

* src/ui/glyphs-eimage.c (rawrgb_instantiate): Only instantiate
image if the data was properly transcoded.

* src/ui/glyphs-eimage.c (png_instantiate): Ditto. In this case,
signal_image_error is best due to complexity.

* src/ui/glyphs-eimage.c (gif_instantiate): Ditto.

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

Coverity: UNINIT: CID 410

* src/ui/glyphs-eimage.c (rawrgba_instantiate): Make sure data is
only used if conversion was proper.

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

Coverity: UNREACHABLE: CID 426, 425, 424, 423, 422, 421

* src/openssl.c (ossl_digest_fun): Just appease Coverity and add
do the while(0)...
* src/openssl.c (ossl_cipher_fun): Ditto.
* src/openssl.c (OSSL_CHECK_PROCESS): Ditto.

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

Coverity: UNINIT: CID 453

* src/ui/glyphs-eimage.c (tiff_instantiate): Make sure file is
properly encoded before using it. This one is a bit uglier due to
the custom unwind here for error handling.

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

Coverity: Uninit: CID 473

* src/media/media-sndfile.c (media_sndfile_open): Make sure to
only call sf_open if file name was properly converted.

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

Coverity: Char IO: CID 595

* lib-src/make-docfile.c (get_dyna_doc): Make sure to properly
downcast the result of getc to char, after checking for its
validity (ie != -1).

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

Coverity: No effect: CID 598

* src/media/media-mad.c (media_mad_open): Use long for result of
ftell. Previous use of size_t could cause a bad ftell to be
considered a maximum size media file... Later the positive size has
to be cast to size_t for storage in the structure.

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

Coverity: LOCK: CID 597

* src/media/sound-esd.c (sound_esd_play): Add missing unlock

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

Merge branch 'libsrc-culling'

* libsrc-culling:
  Get rid of crusty old stuff from lib-src.

Get rid of crusty old stuff from lib-src.

We inherited a boat load of stuff in lib-src when we forked from XEmacs,
it's high time we got rid of all that cave-man bit-rot.  This changeset
gets rid of the stuff in lib-src that nobody under the age of 30 has even
heard of, let alone ever used.

* lib-src/Makefile.am (dist_bin_SCRIPTS): Removed.

* lib-src/Makefile.am (dist_archlib_SCRIPTS): Removed.

* lib-src/Makefile.am (stage1): Remove $dist_bin_SCRIPTS.

* etc/Makefile.am (dist_manone_DATA): Remove gnuattach.1,
gnudoit.1.

* etc/gnuserv.1: Remove mention of gnudoit/gnuattach

* lib-src/gnuserv.c (permitted): Don't call it 'gnudoit' in error
string, it's 'gnuclient'.

* lib-src/gnuserv.c (handle_internet_request): Ditto.

* lib-src/gnuclient.c (pass_signal_to_emacs): Call it 'gnuclient'
in error string.

* lisp/gnuserv.el: Remove mention of gnudoit.

* lisp/menubar-items.el (default-menubar): Call it gnuclient for
remote Emacs-W3 entry.

* etc/README: Remove mention of gnudoit, gnuattach

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

Contrib git conveniences from Nelson

* merges:
  Update README. Give novice advice in git-to-steve.sh
  Add the sample git post-receive hook and update README
  Add clear copyright notices to the scripts I contributed
  Fix build

Update README. Give novice advice in git-to-steve.sh

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

Add the sample git post-receive hook and update README

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

Add clear copyright notices to the scripts I contributed

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

Merge branch 'master' of git.sxemacs.org/sxemacs into for-steve

Merge branch 'trivial'

* trivial:
  missing paren

missing paren

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

Fix build

* src/media/media-sox.c (media_sox_open): Fix stupid typo

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

Coverity fixes from Nelson

* merges:
  Coverity: UNINIT: CID 474
  Coverity: DEADCODE: CID 578
  Coverity: Checked return: CID 582
  Coverity: Missing lock: CID 584
  Coverity: Negative Returns: CID 585
  Coverity: UNINIT: CID 592
  Coverity: NO_EFFECT: CID 594
  Coverity: UNINIT: CID 593 Make sure file name is not used uninitialized.  : Negative Returns: CID 586: Make sure file size is proper

Coverity: UNINIT: CID 474

* src/media/media-sox.c: make sure file conversion is proper

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

Coverity: DEADCODE: CID 578

* src/print.c (SXE_VSNPRINT_VA): Fix the macro so that error in
snprintf is actually detected, instead of masked because of the
buffer growth loop.

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

Coverity: Checked return: CID 582

* src/media/media-internal.c (parsewave): Check the return of
waverequire, because we are trying to align the stream, but it may
still be fatal. Also fix the "flow" through of fatal error parsing
WAVE file in case wvSoundChuck which do not sound proper.

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

Coverity: Missing lock: CID 584

* src/media/sound-esd.c (sound_esd_play): Surround play_state
change with the necessary lock.

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

Coverity: Negative Returns: CID 585

* src/media/media-internal.c (media_internal_analyse_stream): Only
close the fd if it is valid, ie >= 0.

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

Coverity: UNINIT: CID 592

* src/media/media-internal.c (media_internal_analyse_stream): Make
sure that only if conversion to external format is successful we
try to open the file otherwise return as if file open failed.

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

Coverity: NO_EFFECT: CID 594

* lib-src/gnuslib.c (connect_to_unix_server): Use int for sz which
is the proper return type of snprintf.

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

Coverity: UNINIT: CID 593 Make sure file name is not used uninitialized.
: Negative Returns: CID 586: Make sure file size is proper

* src/media/media-mad.c (media_mad_open): Guard fopen against bad
EXTERNAL_FORMAT conversion.
Also make sure that if ftell is not successful the function terminates

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

Coverity: CHAR_IO: CID 595

* lib-src/make-docfile.c (get_dyna_doc): Make explicit the discard
of non existing bits on the getc call assignment to the saved
string.

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

Coverity fixes from Nelson

* merges:
  Coverity: NEGATIVE_RETURNS: CID 588
  Coverity: NULL returns: CID 589
  Coverity: Resource leak: CID 590
  Coverity: Resource leak: CID 591

Coverity: NEGATIVE_RETURNS: CID 588

* src/ui/TTY/gpmevent.c (restore_gpm_state): Make sure that fd is
>= 0 before accesing the state array.

* src/ui/TTY/gpmevent.c (store_gpm_state): Ditto.

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

Coverity: NULL returns: CID 589

* src/process-unix.c (allocate_pty): Make sure slave_name is not
NULL, since that is a possible return from ttyname.

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

Coverity: Resource leak: CID 590

* src/media/media-internal.c (media_internal_analyse_stream): free
data in the case of failed open.

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

Coverity: Resource leak: CID 591

* src/media/media-mad.c (media_mad_open): free sd on unknown driver.

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

Compiler Warning fixes from Nelson

* merges:
  Fix switch warnings in xlwscrollbar
  Fix switch warnings in media
  Remove warnings

Fix switch warnings in xlwscrollbar

* src/ui/lwlib/xlwscrollbar.c (call_callbacks): Add default to switch.
* src/ui/lwlib/xlwscrollbar.c (timer): Init reason to
XmCR_NONE. Assert that ARM_NONE cannot happen in switch.

* src/ui/lwlib/xlwscrollbar.c (Select): Add fall-through case for
BUTTON_NONE. Add default case for another switch.

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

Fix switch warnings in media

* src/media/media.c: New Qunavailable symbol.

* src/media/media.c (Fmedia_properties): Default driver to
Qunavailable. Add case labels for Xine and Gstreamer.

* src/media/media.c (syms_of_media): init Qunavailable.

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

Remove warnings

* src/regex.c (POP_FAILURE_POINT): Move DEBUG_STATEMENT to comply
with C variable declaration rules. Cast the POP_FAILURE_POINTER return
to proper const-ness.

* src/regex.c (re_match_2_internal): Make re_char *'s d and dend
const. Make scope p2 char* const. Make pdummy and sdummy const.
Remove old comment.

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

Improved assertions from Nelson

* merges:
  Improved assert messages

Merge branch 'master' of ssh://dio.dreamhost.com/~/repos.nelsonferreira.com/git/sxemacs into for-steve

Coverity fixes from Nelson

* merges:
  Coverity: Reverse INULL: CID 204
  Coverity: Reverse INULL: CID:207
  Coverity: Reverse Negative: CID:209
  COVERITY: Forward NULL: CID:40
  COVERITY: Forward NULL CID:46
  COVERITY: SECURE_TEMP - CID:345
  Small docstring update

Merge branch 'njsf-cov' into for-steve

Coverity: Reverse INULL: CID 204

* modules/ase/ase-interval.c (_ase_unite_intervals_intr): Defer
the call to ase_cartesion_dimension so that it does not try to
derefence an invalid argument. As a side benefit, it will be
faster in the trivial cases.

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

Coverity: Reverse INULL: CID:207

* src/media/media.c (media_stream_finalise): Return immediately if the stream is NULL

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

Coverity: Reverse Negative: CID:209

* src/mule/file-coding.c (coding_system_from_mask): Make sure cat
is >=0 before dereferencing in the loop.

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

COVERITY: Forward NULL: CID:40

* lib-src/ootags.c (analyse_regex): Return if regex_arg is NULL

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

Improved assert messages

* src/emacs.c: make the assertions_dont_abort a variable instead
of define.

* src/emacs.c (assert_failed): Make the 3rd nested assertion abort
instead of exit. Add an extra newline in the 2nd nested
assertion. Add a short backtrace of up to 32767 entries to the 2nd
nested assertion. Add a backtrace to to the 1st assert_failed.

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

Merge branch 'master' of ssh://dio.dreamhost.com/~/repos.nelsonferreira.com/git/sxemacs into for-steve

Merge branch 'njsf-cov' into for-steve

COVERITY: Forward NULL CID:46

* src/print.c (write_string_to_stdio_stream): Add appropriate
diagnostics if no console nor stream provided...
* src/print.c (Fexternal_debugging_output): Explict NULL
initialization.

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

COVERITY: SECURE_TEMP - CID:345

* lib-src/movemail.c (lock_dot): Force umask to a sensible default, ie remove all group and other permissions

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