Bootloader失败错误

在使用I2C接口时，我找到了一个崩溃引导程序的方法。事实上，这将影响任何引导加载程序。CubTrdRuleCurdDyDATA命令不限制用户通过数据绑定边界追加数据。这会导致意想不到的行为。在我的I2C引导加载程序的情况下，I2C从所有其他传输中获取数据，防止其执行任何命令。虽然这可以被认为是一个用户错误，但结果是一个完全失败，这对于Bootloader是不可接受的。
原代码：
数据实例：
我们有数据块的一部分。*/
ACKCODE＝CyrReTe成功；
MEMCPY（＆DATABOFFER）[DATAOPEN]，和PACETBase[CYBTLDRYDATAYADDR]，PKTSIZE；
DATAOPEN++PKTSIZE；
断裂；
新代码：
数据实例：
我们有数据块的一部分。*/
ACKCODE＝CyrReTe成功；
如果（（DATAOPEN+PKTSIZE）& lsisiffOfCuthdId缓冲区）
{
MEMCPY（＆DATABOFFER）[DATAOPEN]，和PACETBase[CYBTLDRYDATAYADDR]，PKTSIZE；
DATAOPEN++PKTSIZE；
}
其他的
{
ACKCODE＝CyrthyeleLe长度；
}
断裂；



     以下为原文
  I found a way to crash the bootloader when using the I2C interface.  In fact, this should affect any bootloader.  The CYBTLDR_COMMAND_DATA command does not restrict the user from appending data past the dataBuffer boundary.  This will cause unexpected behavior.  In the case of my I2C bootloader, I2C NAKs data from all further transfers, preventing it from performing any command.  Although this can be considered an user error, but the result is a complete failure, which is unacceptable for a bootloader.
     
     
    Original code:
     
    case CYBTLDR_COMMAND_DATA:
                        /* We have part of a block of data. */
                        ackCode = CYRET_SUCCESS;
                        memcpy(&dataBuffer[dataOffset], &packetBuffer[CYBTLDR_DATA_ADDR], pktSize);
                        dataOffset += pktSize;
                        break;
     
    New code:
     
    case CYBTLDR_COMMAND_DATA:
                        /* We have part of a block of data. */
                        ackCode = CYRET_SUCCESS;
                        if ( (dataOffset + pktSize) < SIZEOF_COMMAND_BUFFER )
                        {
                            memcpy(&dataBuffer[dataOffset], &packetBuffer[CYBTLDR_DATA_ADDR], pktSize);
                            dataOffset += pktSize;
                        }
                        else
                        {
                            ackCode = CYRET_ERR_LENGTH;
                        }
                        
                        break;