On Thu, Sep 27, 2012 at 01:31:53AM +0200, Petr Machata wrote:
> [email protected] writes:
> > @@ -131,6 +163,9 @@ arch_elf_init(struct ltelf *lte, struct library *lib)
> >             }

...

> > +       || breakpoint_init(bp, proc,
> > +                          (void *) (uintptr_t) libsym->arch.resolved_addr,
> 
> The double cast shouldn't be necessary here.  uintptr_t already is
> void*, and when we convert it to integral, breakpoint_init will be
> adjusted as well.

Getting back to this, I realized the cast to void * was wrong cause
breakpoint_init want's an arch_addr_t. I've rewritten that specific
part in v2, please take a look at it and see if it looks OK.

Thanks,
Edgar

_______________________________________________
Ltrace-devel mailing list
[email protected]
http://lists.alioth.debian.org/cgi-bin/mailman/listinfo/ltrace-devel

Reply via email to