updown logic error