[email protected] writes:

> +int
> +arch_atomic_singlestep(struct Process *proc, struct breakpoint *sbp,
> +                    int (*add_cb)(void *addr, void *data),
> +                    void *add_cb_data)
> +{
> +     uint32_t pc = (uint32_t) get_instruction_pointer(proc);
> +     uint32_t newpcs[2];
> +     int nr;
> +
> +     nr = mips_next_pcs(proc, pc, newpcs);
> +
> +     while (nr--) {

This should be (nr-- > 0).

> +             arch_addr_t baddr = (arch_addr_t) newpcs[nr];
> +             /* Not sure what to do here. We've already got a bp?  */
> +             if (dict_find_entry(proc->breakpoints, baddr)) {

This should be != NULL.

Thanks,
PM

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

Reply via email to