* doc.c: Integer and memory overflow fixes.
[bpt/emacs.git] / src / ChangeLog
index 4a9e03d..6cf9a1f 100644 (file)
@@ -1,5 +1,83 @@
+2011-07-28  Paul Eggert  <eggert@cs.ucla.edu>
+
+       * doc.c: Integer and memory overflow fixes.
+       (get_doc_string_buffer_size): Now ptrdiff_t, not int.
+       (get_doc_string): Check for size calculation overflow.
+       Don't update size until allocation succeeds.
+       (get_doc_string, Fsubstitute_command_keys): Use ptrdiff_t, not
+       EMACS_INT, where ptrdiff_t will do.
+       (Fsubstitute_command_keys): Check for string overflow.
+
+       Integer and memory overflow fixes for display code.
+       * dispextern.h (struct glyph_pool.nglyphs): Now ptrdiff_t, not int.
+       * dispnew.c (adjust_glyph_matrix, realloc_glyph_pool, scrolling_window):
+       Check for overflow in size calculations.
+       (line_draw_cost, realloc_glyph_pool, add_row_entry):
+       Don't assume glyph table len fits in int.
+       (struct row_entry.bucket, row_entry_pool_size, row_entry_idx)
+       (row_table_size): Now ptrdiff_t, not int.
+       (scrolling_window): Avoid overflow in size calculations.
+       Don't update size until allocation succeeds.
+       * fns.c (concat): Check for overflow in size calculations.
+       (next_almost_prime): Verify NEXT_ALMOST_PRIME_LIMIT.
+       * lisp.h (RANGED_INTEGERP, TYPE_RANGED_INTEGERP): New macros.
+       (NEXT_ALMOST_PRIME_LIMIT): New constant.
+
+       * composite.c: Integer overflow fixes.
+       (get_composition_id): Check for overflow in glyph length calculations.
+
+       * coding.c: Integer and memory overflow fixes.
+       (produce_chars): Redo buffer-overflow calculations to avoid
+       unnecessary integer overflow.  Check for size overflow.
+       (encode_coding_object): Don't update size until xmalloc succeeds.
+
+       * character.c (Fstring): Check for size-calculation overflow.
+
+       * ccl.c: Integer and memory overflow fixes.
+       (Fccl_execute_on_string): Check for memory overflow.
+       Use ptrdiff_t rather than EMACS_INT where ptrdiff_t will do.
+       Redo buffer-overflow calculations to avoid integer overflow.
+
+       * callproc.c (child_setup): Don't assume strlen fits in int.
+
+       * buffer.c: Memory overflow fixes.
+       (overlays_at, overlays_in, record_overlay_string, overlay_strings):
+       Don't update size of array until after memory allocation succeeds,
+       because xmalloc/xrealloc may not return.
+
+       * bidi.c: Integer overflow fix.
+       (bidi_shelve_header_size): New constant.
+       (bidi_cache_ensure_space, bidi_shelve_cache): Use it.
+       (bidi_cache_ensure_space): Avoid integer overflow when allocating.
+
 2011-07-19  Paul Eggert  <eggert@cs.ucla.edu>
 
+       Use ptrdiff_t for composition IDs.
+       * character.c (lisp_string_width):
+       * composite.c (composition_table_size, n_compositions)
+       (get_composition_id, composition_gstring_from_id):
+       * dispextern.h (struct glyph_string.cmp_id, struct composition_it.id):
+       * xdisp.c (BUILD_COMPOSITE_GLYPH_STRING):
+       * window.c (Frecenter):
+       Use ptrdiff_t, not int, for composition IDs.
+       * composite.c (get_composition_id): Check for integer overflow.
+       * composite.h: Adjust prototypes to match the above changes.
+
+       Use ptrdiff_t for hash table indexes.
+       * category.c (hash_get_category_set):
+       * ccl.c (ccl_driver):
+       * charset.h (struct charset.hash_index, CHECK_CHARSET_GET_ID):
+       * coding.c (coding_system_charset_list, detect_coding_system):
+       * coding.h (struct coding_system.id):
+       * composite.c (get_composition_id, gstring_lookup_cache):
+       * fns.c (hash_lookup, hash_put, Fgethash, Fputhash):
+       * image.c (xpm_get_color_table_h):
+       * lisp.h (hash_lookup, hash_put):
+       * minibuf.c (Ftest_completion):
+       Use ptrdiff_t for hash table indexes, not int (which is too
+       narrow, on 64-bit hosts) or EMACS_INT (which is too wide, on
+       32-bit --with-wide-int hosts).
+
        * charset.c (Fdefine_charset_internal): Check for integer overflow.
        Add a FIXME comment about memory leaks.
        (syms_of_charset): Don't assume xmalloc returns.
@@ -14,8 +92,6 @@
        now that we have two such functions.  All uses changed.
        (sanitize_char_width): New inline function.
 
-2011-07-18  Paul Eggert  <eggert@cs.ucla.edu>
-
        Don't assume that tab-width fits in int.
        * character.h (sanitize_width): New inline function.
        (SANE_TAB_WIDTH): New macro.
        Remove unreachable code.
        (read_hex, load_charset_map_from_file): Check for integer overflow.
 
-2011-07-17  Paul Eggert  <eggert@cs.ucla.edu>
-
        * xterm.c: don't go over XClientMessageEvent limit
        (scroll_bar_windows_size): Now ptrdiff_t, as we prefer signed.
        (x_send_scroll_bar_event): Likewise.  Check that the size does not
        (gs_load): Use printmax_t to print the widest integers possible.
        Check for integer overflow when computing image height and width.
 
-2011-07-17  Paul Eggert  <eggert@cs.ucla.edu>
+2011-07-19  Paul Eggert  <eggert@cs.ucla.edu>
 
        Integer signedness and overflow and related fixes.  (Bug#9079)
 
        Use EMACS_INT, not EMACS_UINT, for sizes.  The code works equally
        well either way, and we prefer signed to unsigned.
 
+2011-07-19  Paul Eggert  <eggert@cs.ucla.edu>
+
+       Port to OpenBSD.
+       See http://lists.gnu.org/archive/html/emacs-devel/2011-07/msg00688.html
+       and the surrounding thread.
+       * minibuf.c (read_minibuf_noninteractive): Rewrite to use getchar
+       rather than fgets, and retry after EINTR.  Otherwise, 'emacs
+       --batch -f byte-compile-file' fails on OpenBSD if an inactivity
+       timer goes off.
+       * s/openbsd.h (BROKEN_SIGIO): Define.
+       * unexelf.c (unexec) [__OpenBSD__]:
+       Don't update the .mdebug section of the Alpha COFF symbol table.
+
+2011-07-19  Lars Magne Ingebrigtsen  <larsi@gnus.org>
+
+       * lread.c (syms_of_lread): Clarify when `lexical-binding' is used
+       (bug#8460).
+
+2011-07-18  Paul Eggert  <eggert@cs.ucla.edu>
+
+       * fileio.c (Fcopy_file) [!MSDOS]: Tighten created file's mask.
+       This fixes some race conditions on the permissions of any newly
+       created file.
+
+       * alloc.c (valid_pointer_p): Use pipe, not open.
+       This fixes some permissions issues when debugging.
+
+       * fileio.c (Fcopy_file): Adjust mode if fchown fails.  (Bug#9002)
+       If fchown fails to set both uid and gid, try to set just gid,
+       as that is sometimes allowed.  Adjust the file's mode to eliminate
+       setuid or setgid bits that are inappropriate if fchown fails.
+
+2011-07-18  Stefan Monnier  <monnier@iro.umontreal.ca>
+
+       * xdisp.c (next_element_from_string, next_element_from_buffer): Use EQ
+       to compare Lisp_Objects.
+       * gnutls.c (syms_of_gnutls): Rename Vgnutls_log_level to
+       global_gnutls_log_level, don't mistake it for a Lisp_Object.
+       (init_gnutls_functions, emacs_gnutls_handle_error): Fix up uses.
+
+2011-07-17  Andreas Schwab  <schwab@linux-m68k.org>
+
+       * lread.c (read_integer): Unread even EOF character.
+       (read1): Likewise.  Properly record start position of symbol.
+
+       * lread.c (read1): Read `#:' as empty uninterned symbol if no
+       symbol character follows.
+
 2011-07-17  Paul Eggert  <eggert@cs.ucla.edu>
 
        * fileio.c (Fcopy_file): Pacify gcc re fchown.  (Bug#9002)