logic error fix#