Keypad

A micro-controller with pins PB0-PB6 is connected to the 7 pins of a keypad where 4 rows (output) connect to PB3-PB6 and 3 columns (input) to PB0-PB2. The input column bits will only be set to high after a key is pressed from a row that is set high at the time (keys are like circuit switches). To find which key has been pressed, rows are sequentially set high so if a key is pressed on a high row a column bit will be set high. I have LEDs connected to the rows, until a key is pressed all lights appear lit because they are being iterated through so fast. My objective is to terminate the program after a key is pressed leaving the light matching the row bit high. Problem is this code is not leaving the expected light on so it must be terminating with the wrong row bit set high.

What i am attempting to do with the code is shift through each bit of PB3-6, at each row bit shift, all column bits are shifted and the carry bit is checked to see if a key is pressed and terminated if so. I would like to know if this code is not doing what i'm describing here.

not clear: attach an image with schematic please, at least pins, leds and keys. and what MCU are you using exactly? in any case it is better to scan with shifting zero then shifting high level (high level on output is inactive while on zero level the current goes into the ground through MCU pin)