public inbox for frysk-bugzilla@sourceware.org
help / color / mirror / Atom feed
* [Bug general/4318] New: ExprSymTab uses signed long for length
@ 2007-04-04  9:49 mark at klomp dot org
  2007-04-04 15:09 ` [Bug general/4318] " pmuldoon at redhat dot com
                   ` (5 more replies)
  0 siblings, 6 replies; 7+ messages in thread
From: mark at klomp dot org @ 2007-04-04  9:49 UTC (permalink / raw)
  To: frysk-bugzilla

// ??? 0x7fffffffffffffff
    buffer = new PtraceByteBuffer(task.getTid(), PtraceByteBuffer.Area.DATA,
                                  0x7fffffffffffffffl);

That should most likely be -1L (unsigned long 0xffffffffffffffff)

-- 
           Summary: ExprSymTab uses signed long for length
           Product: frysk
           Version: unspecified
            Status: NEW
          Severity: normal
          Priority: P2
         Component: general
        AssignedTo: frysk-bugzilla at sourceware dot org
        ReportedBy: mark at klomp dot org


http://sourceware.org/bugzilla/show_bug.cgi?id=4318

------- You are receiving this mail because: -------
You are the assignee for the bug, or are watching the assignee.


^ permalink raw reply	[flat|nested] 7+ messages in thread

* [Bug general/4318] ExprSymTab uses signed long for length
  2007-04-04  9:49 [Bug general/4318] New: ExprSymTab uses signed long for length mark at klomp dot org
@ 2007-04-04 15:09 ` pmuldoon at redhat dot com
  2007-04-04 15:41 ` cagney at redhat dot com
                   ` (4 subsequent siblings)
  5 siblings, 0 replies; 7+ messages in thread
From: pmuldoon at redhat dot com @ 2007-04-04 15:09 UTC (permalink / raw)
  To: frysk-bugzilla


------- Additional Comments From pmuldoon at redhat dot com  2007-04-04 16:09 -------
Any reason why it isn't using task.getMemory() instead of mapping another
PtraceByteBuffer?

-- 


http://sourceware.org/bugzilla/show_bug.cgi?id=4318

------- You are receiving this mail because: -------
You are the assignee for the bug, or are watching the assignee.


^ permalink raw reply	[flat|nested] 7+ messages in thread

* [Bug general/4318] ExprSymTab uses signed long for length
  2007-04-04  9:49 [Bug general/4318] New: ExprSymTab uses signed long for length mark at klomp dot org
  2007-04-04 15:09 ` [Bug general/4318] " pmuldoon at redhat dot com
@ 2007-04-04 15:41 ` cagney at redhat dot com
  2007-04-10 19:35 ` pmuldoon at redhat dot com
                   ` (3 subsequent siblings)
  5 siblings, 0 replies; 7+ messages in thread
From: cagney at redhat dot com @ 2007-04-04 15:41 UTC (permalink / raw)
  To: frysk-bugzilla


------- Additional Comments From cagney at redhat dot com  2007-04-04 16:40 -------
(In reply to comment #1)
> Any reason why it isn't using task.getMemory() instead of mapping another
> PtraceByteBuffer?

Yes, for corefile, and other Proc's it must use task.getMemory().

-- 


http://sourceware.org/bugzilla/show_bug.cgi?id=4318

------- You are receiving this mail because: -------
You are the assignee for the bug, or are watching the assignee.


^ permalink raw reply	[flat|nested] 7+ messages in thread

* [Bug general/4318] ExprSymTab uses signed long for length
  2007-04-04  9:49 [Bug general/4318] New: ExprSymTab uses signed long for length mark at klomp dot org
  2007-04-04 15:09 ` [Bug general/4318] " pmuldoon at redhat dot com
  2007-04-04 15:41 ` cagney at redhat dot com
@ 2007-04-10 19:35 ` pmuldoon at redhat dot com
  2007-04-10 22:01 ` cagney at redhat dot com
                   ` (2 subsequent siblings)
  5 siblings, 0 replies; 7+ messages in thread
From: pmuldoon at redhat dot com @ 2007-04-10 19:35 UTC (permalink / raw)
  To: frysk-bugzilla


------- Additional Comments From pmuldoon at redhat dot com  2007-04-10 20:35 -------
Created an attachment (id=1671)
 --> (http://sourceware.org/bugzilla/attachment.cgi?id=1671&action=view)
Expression task memory access

Patch to abstract memory access to using task.getMemory() instead of mapping a
PtraceByteBuffer directly (one cannot assume the task is a Ptrace Task)

-- 


http://sourceware.org/bugzilla/show_bug.cgi?id=4318

------- You are receiving this mail because: -------
You are the assignee for the bug, or are watching the assignee.


^ permalink raw reply	[flat|nested] 7+ messages in thread

* [Bug general/4318] ExprSymTab uses signed long for length
  2007-04-04  9:49 [Bug general/4318] New: ExprSymTab uses signed long for length mark at klomp dot org
                   ` (2 preceding siblings ...)
  2007-04-10 19:35 ` pmuldoon at redhat dot com
@ 2007-04-10 22:01 ` cagney at redhat dot com
  2007-04-10 22:34 ` pmuldoon at redhat dot com
  2007-04-11  2:51 ` cagney at redhat dot com
  5 siblings, 0 replies; 7+ messages in thread
From: cagney at redhat dot com @ 2007-04-10 22:01 UTC (permalink / raw)
  To: frysk-bugzilla


------- Additional Comments From cagney at redhat dot com  2007-04-10 23:01 -------
Just fix it.

-- 
           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |RESOLVED
         Resolution|                            |FIXED


http://sourceware.org/bugzilla/show_bug.cgi?id=4318

------- You are receiving this mail because: -------
You are the assignee for the bug, or are watching the assignee.


^ permalink raw reply	[flat|nested] 7+ messages in thread

* [Bug general/4318] ExprSymTab uses signed long for length
  2007-04-04  9:49 [Bug general/4318] New: ExprSymTab uses signed long for length mark at klomp dot org
                   ` (3 preceding siblings ...)
  2007-04-10 22:01 ` cagney at redhat dot com
@ 2007-04-10 22:34 ` pmuldoon at redhat dot com
  2007-04-11  2:51 ` cagney at redhat dot com
  5 siblings, 0 replies; 7+ messages in thread
From: pmuldoon at redhat dot com @ 2007-04-10 22:34 UTC (permalink / raw)
  To: frysk-bugzilla


------- Additional Comments From pmuldoon at redhat dot com  2007-04-10 23:34 -------
Don't know if this fix is appropriate. I'll commit the patch later on tonight,
and run tests then.

-- 
           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|RESOLVED                    |REOPENED
         Resolution|FIXED                       |


http://sourceware.org/bugzilla/show_bug.cgi?id=4318

------- You are receiving this mail because: -------
You are the assignee for the bug, or are watching the assignee.


^ permalink raw reply	[flat|nested] 7+ messages in thread

* [Bug general/4318] ExprSymTab uses signed long for length
  2007-04-04  9:49 [Bug general/4318] New: ExprSymTab uses signed long for length mark at klomp dot org
                   ` (4 preceding siblings ...)
  2007-04-10 22:34 ` pmuldoon at redhat dot com
@ 2007-04-11  2:51 ` cagney at redhat dot com
  5 siblings, 0 replies; 7+ messages in thread
From: cagney at redhat dot com @ 2007-04-11  2:51 UTC (permalink / raw)
  To: frysk-bugzilla


------- Additional Comments From cagney at redhat dot com  2007-04-11 03:51 -------
I've already tested and committed that change.  I independantly stumbled across
it while making ptrace changes.

-- 
           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|REOPENED                    |RESOLVED
         Resolution|                            |FIXED


http://sourceware.org/bugzilla/show_bug.cgi?id=4318

------- You are receiving this mail because: -------
You are the assignee for the bug, or are watching the assignee.


^ permalink raw reply	[flat|nested] 7+ messages in thread

end of thread, other threads:[~2007-04-11  2:51 UTC | newest]

Thread overview: 7+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2007-04-04  9:49 [Bug general/4318] New: ExprSymTab uses signed long for length mark at klomp dot org
2007-04-04 15:09 ` [Bug general/4318] " pmuldoon at redhat dot com
2007-04-04 15:41 ` cagney at redhat dot com
2007-04-10 19:35 ` pmuldoon at redhat dot com
2007-04-10 22:01 ` cagney at redhat dot com
2007-04-10 22:34 ` pmuldoon at redhat dot com
2007-04-11  2:51 ` cagney at redhat dot com

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).