From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 7196 invoked by alias); 5 Jun 2012 21:23:22 -0000 Received: (qmail 7188 invoked by uid 22791); 5 Jun 2012 21:23:21 -0000 X-SWARE-Spam-Status: No, hits=-6.2 required=5.0 tests=AWL,BAYES_00,KHOP_RCVD_UNTRUST,RCVD_IN_DNSWL_HI,RCVD_IN_HOSTKARMA_W,SPF_HELO_PASS,TW_YM,T_RP_MATCHES_RCVD X-Spam-Check-By: sourceware.org Received: from mx1.redhat.com (HELO mx1.redhat.com) (209.132.183.28) by sourceware.org (qpsmtpd/0.43rc1) with ESMTP; Tue, 05 Jun 2012 21:23:07 +0000 Received: from int-mx12.intmail.prod.int.phx2.redhat.com (int-mx12.intmail.prod.int.phx2.redhat.com [10.5.11.25]) by mx1.redhat.com (8.14.4/8.14.4) with ESMTP id q55LN69V013896 (version=TLSv1/SSLv3 cipher=DHE-RSA-AES256-SHA bits=256 verify=OK) for ; Tue, 5 Jun 2012 17:23:07 -0400 Received: from host2.jankratochvil.net (ovpn-116-47.ams2.redhat.com [10.36.116.47]) by int-mx12.intmail.prod.int.phx2.redhat.com (8.14.4/8.14.4) with ESMTP id q55LN1mW022409 (version=TLSv1/SSLv3 cipher=DHE-RSA-AES128-SHA bits=128 verify=NO); Tue, 5 Jun 2012 17:23:03 -0400 Date: Tue, 05 Jun 2012 21:23:00 -0000 From: Jan Kratochvil To: Siddhesh Poyarekar Cc: gdb-patches@sourceware.org, Tom Tromey Subject: [commit] [PATCH] Memory reads and writes should have size_t length Message-ID: <20120605212300.GB31768@host2.jankratochvil.net> References: <20120531125320.65ad1f8f@spoyarek> <20120601174809.GA21938@host2.jankratochvil.net> <20120602012958.4a6d9a7c@spoyarek> <20120604215419.GA27980@host2.jankratochvil.net> <20120605084605.186d537a@spoyarek> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20120605084605.186d537a@spoyarek> User-Agent: Mutt/1.5.21 (2010-09-15) X-IsSubscribed: yes Mailing-List: contact gdb-patches-help@sourceware.org; run by ezmlm Precedence: bulk List-Id: List-Subscribe: List-Archive: List-Post: List-Help: , Sender: gdb-patches-owner@sourceware.org X-SW-Source: 2012-06/txt/msg00160.txt.bz2 On Tue, 05 Jun 2012 05:16:05 +0200, Siddhesh Poyarekar wrote: > OK, thanks for the explanation. Updated patch attached. Checked in. (simplified ChangeLog) tested on {x86_64,x86_64-m32}-fedora18pre-linux-gnu. Thanks, Jan http://sourceware.org/ml/gdb-cvs/2012-06/msg00038.html --- src/gdb/ChangeLog 2012/06/05 21:18:04 1.14328 +++ src/gdb/ChangeLog 2012/06/05 21:22:30 1.14329 @@ -1,3 +1,14 @@ +2012-06-05 Siddhesh Poyarekar + + * corefile.c (read_memory, read_stack, write_memory): Accept LEN + argument as ssize_t. + * gdbcore.h (read_memory, read_stack, write_memory): Likewise. + * remote.c (remote_write_bytes_aux, remote_write_bytes): Likewise. + * target.c (target_read_stack, target_write_memory) + (target_write_raw_memory): Likewise. + * target.h (target_read_stack, target_write_memory) + (target_write_raw_memory): Likewise. + 2012-06-05 Jan Kratochvil * symfile-mem.c: Change gdb_static_assert to ssize_t. [patch is the same]