[Patch]: Allow dump memory command to dump more than 2GB

[Patch]: Allow dump memory command to dump more than 2GB

[Sterling Augustine]

[-- Attachment #1: Type: text/plain, Size: 380 bytes --]

Hi All,

The enclosed, basically trivial, patch allows the dump memory command
to dump more than 2GB. It works simply by changing the parameter type
to match the declared type of the caller's argument.

Thanks,

Sterling


2011-07-26  Sterling Augustine  <saugustine@google.com>

	* cli/cli-dump.c (dump_binary_file): Change paramter type to ULONGEST.
	(dump_bfd_file): Likewise.

[-- Attachment #2: dump_large.patch --]
[-- Type: text/x-patch, Size: 735 bytes --]

Index: cli-dump.c
===================================================================
RCS file: /cvs/src/src/gdb/cli/cli-dump.c,v
retrieving revision 1.44
diff -d -u -r1.44 cli-dump.c
--- cli-dump.c	3 Jun 2011 23:47:46 -0000	1.44
+++ cli-dump.c	26 Jul 2011 19:24:46 -0000
@@ -174,7 +174,7 @@
 
 static void
 dump_binary_file (const char *filename, const char *mode, 
-		  const bfd_byte *buf, int len)
+		  const bfd_byte *buf, ULONGEST len)
 {
   FILE *file;
   int status;
@@ -188,7 +188,7 @@
 static void
 dump_bfd_file (const char *filename, const char *mode, 
 	       const char *target, CORE_ADDR vaddr, 
-	       const bfd_byte *buf, int len)
+	       const bfd_byte *buf, ULONGEST len)
 {
   bfd *obfd;
   asection *osection;

Re: [Patch]: Allow dump memory command to dump more than 2GB

[Tom Tromey]

>>>>> "Sterling" == Sterling Augustine <saugustine@google.com> writes:

Sterling> 2011-07-26  Sterling Augustine  <saugustine@google.com>
Sterling> 	* cli/cli-dump.c (dump_binary_file): Change paramter type to ULONGEST.
Sterling> 	(dump_bfd_file): Likewise.

Ok.

Tom

Re: [Patch]: Allow dump memory command to dump more than 2GB

[Paul Pluzhnikov]

On Tue, Jul 26, 2011 at 1:02 PM, Tom Tromey <tromey@redhat.com> wrote:
>
> Sterling> 2011-07-26  Sterling Augustine  <saugustine@google.com>
> Sterling>       * cli/cli-dump.c (dump_binary_file): Change paramter type to ULONGEST.
> Sterling>       (dump_bfd_file): Likewise.
>
> Ok.

I've committed this patch for Sterling.

Thanks,
-- 
Paul Pluzhnikov