From mboxrd@z Thu Jan 1 00:00:00 1970 Path: news.gmane.org!not-for-mail From: Eli Zaretskii Newsgroups: gmane.emacs.bugs Subject: bug#9079: integer overflow etc. issues (e.g., image crashes Emacs) Date: Thu, 14 Jul 2011 04:47:23 -0400 Message-ID: References: <4E1EA0FB.5070903@cs.ucla.edu> Reply-To: Eli Zaretskii NNTP-Posting-Host: lo.gmane.org X-Trace: dough.gmane.org 1310634611 23742 80.91.229.12 (14 Jul 2011 09:10:11 GMT) X-Complaints-To: usenet@dough.gmane.org NNTP-Posting-Date: Thu, 14 Jul 2011 09:10:11 +0000 (UTC) Cc: 9079@debbugs.gnu.org To: Paul Eggert Original-X-From: bug-gnu-emacs-bounces+geb-bug-gnu-emacs=m.gmane.org@gnu.org Thu Jul 14 11:10:07 2011 Return-path: Envelope-to: geb-bug-gnu-emacs@m.gmane.org Original-Received: from lists.gnu.org ([140.186.70.17]) by lo.gmane.org with esmtp (Exim 4.69) (envelope-from ) id 1QhHvq-00046l-Uo for geb-bug-gnu-emacs@m.gmane.org; Thu, 14 Jul 2011 11:10:07 +0200 Original-Received: from localhost ([::1]:43618 helo=lists.gnu.org) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1QhHvp-0002TM-J6 for geb-bug-gnu-emacs@m.gmane.org; Thu, 14 Jul 2011 05:10:05 -0400 Original-Received: from eggs.gnu.org ([140.186.70.92]:43125) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1QhHaa-0005mm-Om for bug-gnu-emacs@gnu.org; Thu, 14 Jul 2011 04:48:13 -0400 Original-Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1QhHaV-00033A-T0 for bug-gnu-emacs@gnu.org; Thu, 14 Jul 2011 04:48:08 -0400 Original-Received: from debbugs.gnu.org ([140.186.70.43]:60874) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1QhHaV-000334-F2 for bug-gnu-emacs@gnu.org; Thu, 14 Jul 2011 04:48:03 -0400 Original-Received: from Debian-debbugs by debbugs.gnu.org with local (Exim 4.69) (envelope-from ) id 1QhHaU-0002Ik-Kp; Thu, 14 Jul 2011 04:48:02 -0400 X-Loop: help-debbugs@gnu.org Resent-From: Eli Zaretskii Original-Sender: debbugs-submit-bounces@debbugs.gnu.org Resent-To: owner@debbugs.gnu.org Resent-CC: bug-gnu-emacs@gnu.org Resent-Date: Thu, 14 Jul 2011 08:48:02 +0000 Resent-Message-ID: Resent-Sender: help-debbugs@gnu.org X-GNU-PR-Message: followup 9079 X-GNU-PR-Package: emacs X-GNU-PR-Keywords: patch Original-Received: via spool by 9079-submit@debbugs.gnu.org id=B9079.13106332538810 (code B ref 9079); Thu, 14 Jul 2011 08:48:02 +0000 Original-Received: (at 9079) by debbugs.gnu.org; 14 Jul 2011 08:47:33 +0000 Original-Received: from localhost ([127.0.0.1] helo=debbugs.gnu.org) by debbugs.gnu.org with esmtp (Exim 4.69) (envelope-from ) id 1QhHa1-0002I3-D4 for submit@debbugs.gnu.org; Thu, 14 Jul 2011 04:47:33 -0400 Original-Received: from fencepost.gnu.org ([140.186.70.10]) by debbugs.gnu.org with esmtp (Exim 4.69) (envelope-from ) id 1QhHZy-0002Hq-CU for 9079@debbugs.gnu.org; Thu, 14 Jul 2011 04:47:31 -0400 Original-Received: from eliz by fencepost.gnu.org with local (Exim 4.71) (envelope-from ) id 1QhHZr-0007YT-29; Thu, 14 Jul 2011 04:47:23 -0400 In-reply-to: <4E1EA0FB.5070903@cs.ucla.edu> (message from Paul Eggert on Thu, 14 Jul 2011 00:55:39 -0700) X-BeenThere: debbugs-submit@debbugs.gnu.org X-Mailman-Version: 2.1.11 Precedence: list Resent-Date: Thu, 14 Jul 2011 04:48:02 -0400 X-detected-operating-system: by eggs.gnu.org: GNU/Linux 2.6 (newer, 3) X-Received-From: 140.186.70.43 X-BeenThere: bug-gnu-emacs@gnu.org List-Id: "Bug reports for GNU Emacs, the Swiss army knife of text editors" List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: bug-gnu-emacs-bounces+geb-bug-gnu-emacs=m.gmane.org@gnu.org Original-Sender: bug-gnu-emacs-bounces+geb-bug-gnu-emacs=m.gmane.org@gnu.org Xref: news.gmane.org gmane.emacs.bugs:48983 Archived-At: > Date: Thu, 14 Jul 2011 00:55:39 -0700 > From: Paul Eggert > > I examined more of the Emacs C source code, looking for problems with > integer overflow and signedness issues, and found several issues, > all fixed in the patch below. Thanks, but please don't install the changes to bidi.c. That file is changed considerably on my local branch, as part of my work on bidi reordering of display string, and I plan to merge it with the trunk in a day or two. Your changes will just get in the way of the merge, which currently holds off the pretest. Once the merge is done, I will see if any of your changes are still applicable, and install them then. TIA