Skip to content

Commit

Permalink
[ARM] 2959/1: Add test for invalid LDRD/STRD Rd cases in ARM alignmen…
Browse files Browse the repository at this point in the history
…t handler

Patch from George G. Davis

Add test for invalid LDRD/STRD Rd cases in ARM alignment handler
and restore SWP printk KERN_ERR.

Signed-off-by: Steve Longerbeam <slongerbeam@mvista.com>
Signed-off-by: George G. Davis <gdavis@mvista.com>
Signed-off-by: Russell King <rmk+kernel@arm.linux.org.uk>
  • Loading branch information
George G. Davis authored and Russell King committed Oct 10, 2005
1 parent 79d13b6 commit 19da83f
Showing 1 changed file with 10 additions and 1 deletion.
11 changes: 10 additions & 1 deletion arch/arm/mm/alignment.c
Original file line number Diff line number Diff line change
Expand Up @@ -330,6 +330,9 @@ do_alignment_ldrdstrd(unsigned long addr, unsigned long instr,
{
unsigned int rd = RD_BITS(instr);

if (((rd & 1) == 1) || (rd == 14))
goto bad;

ai_dword += 1;

if (user_mode(regs))
Expand Down Expand Up @@ -361,7 +364,8 @@ do_alignment_ldrdstrd(unsigned long addr, unsigned long instr,
}

return TYPE_LDST;

bad:
return TYPE_ERROR;
fault:
return TYPE_FAULT;
}
Expand Down Expand Up @@ -663,6 +667,8 @@ do_alignment(unsigned long addr, unsigned int fsr, struct pt_regs *regs)
else if ((instr & 0x001000f0) == 0x000000d0 || /* LDRD */
(instr & 0x001000f0) == 0x000000f0) /* STRD */
handler = do_alignment_ldrdstrd;
else if ((instr & 0x01f00ff0) == 0x01000090) /* SWP */
goto swp;
else
goto bad;
break;
Expand Down Expand Up @@ -733,6 +739,9 @@ do_alignment(unsigned long addr, unsigned int fsr, struct pt_regs *regs)
do_bad_area(current, current->mm, addr, fsr, regs);
return 0;

swp:
printk(KERN_ERR "Alignment trap: not handling swp instruction\n");

bad:
/*
* Oops, we didn't handle the instruction.
Expand Down

0 comments on commit 19da83f

Please sign in to comment.