From f8783e68fa6c787f1d6d3c24396033bb2092fb5f Mon Sep 17 00:00:00 2001 From: Jacob Hageman Date: Thu, 21 May 2020 11:25:10 -0400 Subject: [PATCH] Fix #16, Apply standard code style --- cfe_ts_crc.c | 209 ++++++++++++++++++++++++--------------------------- 1 file changed, 97 insertions(+), 112 deletions(-) diff --git a/cfe_ts_crc.c b/cfe_ts_crc.c index c0109f0..5209f6d 100644 --- a/cfe_ts_crc.c +++ b/cfe_ts_crc.c @@ -23,20 +23,20 @@ ** algorithm as the LRO spacecraft cFE Table Services flight software uses. ** ** Inputs: One string containing the filename of the file to CRC. -** +** ** ** Outputs: Prints to the terminal the filename, size, and CRC. ** Returns the CRC. ** ** Author: Mike Blau, GSFC Code 582 ** -** Date: 1/28/08 +** Date: 1/28/08 ** ** Modified 4/24/08 MDB Added option to skip a specified number of header bytes ** Modified 2/04/09 BDT Modified to compute cFE table services CS ** Modified 4/01/09 STS Modified to always skip header (116 bytes) ** Modified 4/01/09 STS Removed option to skip a specified number of header bytes -** Modified 6/15/12 WFM Replaced the CRC Table with the table used in +** Modified 6/15/12 WFM Replaced the CRC Table with the table used in ** CFE_ES_CalculateCRC */ #include @@ -45,18 +45,17 @@ #include #include -#define CFE_ES_CRC_8 1 /**< \brief CRC ( 8 bit additive - returns 32 bit total) (Currently not implemented) */ -#define CFE_ES_CRC_16 2 /**< \brief CRC (16 bit additive - returns 32 bit total) */ -#define CFE_ES_CRC_32 3 /**< \brief CRC (32 bit additive - returns 32 bit total) (Currently not implemented) */ -#define CFE_ES_DEFAULT_CRC CFE_ES_CRC_16 /**< \brief mission specific CRC type */ +#define CFE_ES_CRC_8 1 /**< \brief CRC ( 8 bit additive - returns 32 bit total) (Currently not implemented) */ +#define CFE_ES_CRC_16 2 /**< \brief CRC (16 bit additive - returns 32 bit total) */ +#define CFE_ES_CRC_32 3 /**< \brief CRC (32 bit additive - returns 32 bit total) (Currently not implemented) */ +#define CFE_ES_DEFAULT_CRC CFE_ES_CRC_16 /**< \brief mission specific CRC type */ -typedef unsigned char uint8; +typedef unsigned char uint8; typedef unsigned short uint16; -typedef unsigned int uint32; -typedef signed char int8; -typedef signed short int16; -typedef signed int int32; - +typedef unsigned int uint32; +typedef signed char int8; +typedef signed short int16; +typedef signed int int32; /* ** Function Prologue @@ -73,119 +72,105 @@ uint32 CFE_ES_CalculateCRC(void *DataPtr, uint32 DataLength, uint32 InputCRC, ui int16 Crc = 0; uint8 *BufPtr; - static const uint16 CrcTable[256]= - { - - 0x0000, 0xC0C1, 0xC181, 0x0140, 0xC301, 0x03C0, 0x0280, 0xC241, - 0xC601, 0x06C0, 0x0780, 0xC741, 0x0500, 0xC5C1, 0xC481, 0x0440, - 0xCC01, 0x0CC0, 0x0D80, 0xCD41, 0x0F00, 0xCFC1, 0xCE81, 0x0E40, - 0x0A00, 0xCAC1, 0xCB81, 0x0B40, 0xC901, 0x09C0, 0x0880, 0xC841, - 0xD801, 0x18C0, 0x1980, 0xD941, 0x1B00, 0xDBC1, 0xDA81, 0x1A40, - 0x1E00, 0xDEC1, 0xDF81, 0x1F40, 0xDD01, 0x1DC0, 0x1C80, 0xDC41, - 0x1400, 0xD4C1, 0xD581, 0x1540, 0xD701, 0x17C0, 0x1680, 0xD641, - 0xD201, 0x12C0, 0x1380, 0xD341, 0x1100, 0xD1C1, 0xD081, 0x1040, - 0xF001, 0x30C0, 0x3180, 0xF141, 0x3300, 0xF3C1, 0xF281, 0x3240, - 0x3600, 0xF6C1, 0xF781, 0x3740, 0xF501, 0x35C0, 0x3480, 0xF441, - 0x3C00, 0xFCC1, 0xFD81, 0x3D40, 0xFF01, 0x3FC0, 0x3E80, 0xFE41, - 0xFA01, 0x3AC0, 0x3B80, 0xFB41, 0x3900, 0xF9C1, 0xF881, 0x3840, - 0x2800, 0xE8C1, 0xE981, 0x2940, 0xEB01, 0x2BC0, 0x2A80, 0xEA41, - 0xEE01, 0x2EC0, 0x2F80, 0xEF41, 0x2D00, 0xEDC1, 0xEC81, 0x2C40, - 0xE401, 0x24C0, 0x2580, 0xE541, 0x2700, 0xE7C1, 0xE681, 0x2640, - 0x2200, 0xE2C1, 0xE381, 0x2340, 0xE101, 0x21C0, 0x2080, 0xE041, - 0xA001, 0x60C0, 0x6180, 0xA141, 0x6300, 0xA3C1, 0xA281, 0x6240, - 0x6600, 0xA6C1, 0xA781, 0x6740, 0xA501, 0x65C0, 0x6480, 0xA441, - 0x6C00, 0xACC1, 0xAD81, 0x6D40, 0xAF01, 0x6FC0, 0x6E80, 0xAE41, - 0xAA01, 0x6AC0, 0x6B80, 0xAB41, 0x6900, 0xA9C1, 0xA881, 0x6840, - 0x7800, 0xB8C1, 0xB981, 0x7940, 0xBB01, 0x7BC0, 0x7A80, 0xBA41, - 0xBE01, 0x7EC0, 0x7F80, 0xBF41, 0x7D00, 0xBDC1, 0xBC81, 0x7C40, - 0xB401, 0x74C0, 0x7580, 0xB541, 0x7700, 0xB7C1, 0xB681, 0x7640, - 0x7200, 0xB2C1, 0xB381, 0x7340, 0xB101, 0x71C0, 0x7080, 0xB041, - 0x5000, 0x90C1, 0x9181, 0x5140, 0x9301, 0x53C0, 0x5280, 0x9241, - 0x9601, 0x56C0, 0x5780, 0x9741, 0x5500, 0x95C1, 0x9481, 0x5440, - 0x9C01, 0x5CC0, 0x5D80, 0x9D41, 0x5F00, 0x9FC1, 0x9E81, 0x5E40, - 0x5A00, 0x9AC1, 0x9B81, 0x5B40, 0x9901, 0x59C0, 0x5880, 0x9841, - 0x8801, 0x48C0, 0x4980, 0x8941, 0x4B00, 0x8BC1, 0x8A81, 0x4A40, - 0x4E00, 0x8EC1, 0x8F81, 0x4F40, 0x8D01, 0x4DC0, 0x4C80, 0x8C41, - 0x4400, 0x84C1, 0x8581, 0x4540, 0x8701, 0x47C0, 0x4680, 0x8641, - 0x8201, 0x42C0, 0x4380, 0x8341, 0x4100, 0x81C1, 0x8081, 0x4040 - }; - - switch(TypeCRC) + static const uint16 CrcTable[256] = { + + 0x0000, 0xC0C1, 0xC181, 0x0140, 0xC301, 0x03C0, 0x0280, 0xC241, 0xC601, 0x06C0, 0x0780, 0xC741, 0x0500, 0xC5C1, + 0xC481, 0x0440, 0xCC01, 0x0CC0, 0x0D80, 0xCD41, 0x0F00, 0xCFC1, 0xCE81, 0x0E40, 0x0A00, 0xCAC1, 0xCB81, 0x0B40, + 0xC901, 0x09C0, 0x0880, 0xC841, 0xD801, 0x18C0, 0x1980, 0xD941, 0x1B00, 0xDBC1, 0xDA81, 0x1A40, 0x1E00, 0xDEC1, + 0xDF81, 0x1F40, 0xDD01, 0x1DC0, 0x1C80, 0xDC41, 0x1400, 0xD4C1, 0xD581, 0x1540, 0xD701, 0x17C0, 0x1680, 0xD641, + 0xD201, 0x12C0, 0x1380, 0xD341, 0x1100, 0xD1C1, 0xD081, 0x1040, 0xF001, 0x30C0, 0x3180, 0xF141, 0x3300, 0xF3C1, + 0xF281, 0x3240, 0x3600, 0xF6C1, 0xF781, 0x3740, 0xF501, 0x35C0, 0x3480, 0xF441, 0x3C00, 0xFCC1, 0xFD81, 0x3D40, + 0xFF01, 0x3FC0, 0x3E80, 0xFE41, 0xFA01, 0x3AC0, 0x3B80, 0xFB41, 0x3900, 0xF9C1, 0xF881, 0x3840, 0x2800, 0xE8C1, + 0xE981, 0x2940, 0xEB01, 0x2BC0, 0x2A80, 0xEA41, 0xEE01, 0x2EC0, 0x2F80, 0xEF41, 0x2D00, 0xEDC1, 0xEC81, 0x2C40, + 0xE401, 0x24C0, 0x2580, 0xE541, 0x2700, 0xE7C1, 0xE681, 0x2640, 0x2200, 0xE2C1, 0xE381, 0x2340, 0xE101, 0x21C0, + 0x2080, 0xE041, 0xA001, 0x60C0, 0x6180, 0xA141, 0x6300, 0xA3C1, 0xA281, 0x6240, 0x6600, 0xA6C1, 0xA781, 0x6740, + 0xA501, 0x65C0, 0x6480, 0xA441, 0x6C00, 0xACC1, 0xAD81, 0x6D40, 0xAF01, 0x6FC0, 0x6E80, 0xAE41, 0xAA01, 0x6AC0, + 0x6B80, 0xAB41, 0x6900, 0xA9C1, 0xA881, 0x6840, 0x7800, 0xB8C1, 0xB981, 0x7940, 0xBB01, 0x7BC0, 0x7A80, 0xBA41, + 0xBE01, 0x7EC0, 0x7F80, 0xBF41, 0x7D00, 0xBDC1, 0xBC81, 0x7C40, 0xB401, 0x74C0, 0x7580, 0xB541, 0x7700, 0xB7C1, + 0xB681, 0x7640, 0x7200, 0xB2C1, 0xB381, 0x7340, 0xB101, 0x71C0, 0x7080, 0xB041, 0x5000, 0x90C1, 0x9181, 0x5140, + 0x9301, 0x53C0, 0x5280, 0x9241, 0x9601, 0x56C0, 0x5780, 0x9741, 0x5500, 0x95C1, 0x9481, 0x5440, 0x9C01, 0x5CC0, + 0x5D80, 0x9D41, 0x5F00, 0x9FC1, 0x9E81, 0x5E40, 0x5A00, 0x9AC1, 0x9B81, 0x5B40, 0x9901, 0x59C0, 0x5880, 0x9841, + 0x8801, 0x48C0, 0x4980, 0x8941, 0x4B00, 0x8BC1, 0x8A81, 0x4A40, 0x4E00, 0x8EC1, 0x8F81, 0x4F40, 0x8D01, 0x4DC0, + 0x4C80, 0x8C41, 0x4400, 0x84C1, 0x8581, 0x4540, 0x8701, 0x47C0, 0x4680, 0x8641, 0x8201, 0x42C0, 0x4380, 0x8341, + 0x4100, 0x81C1, 0x8081, 0x4040}; + + switch (TypeCRC) { -/* case CFE_ES_CRC_32: */ -/* CFE_ES_WriteToSysLog("CFE ES Calculate CRC32 not Implemented\n"); */ -/* break; */ - - case CFE_ES_CRC_16: - Crc = (int16 )( 0xFFFF & InputCRC ); - BufPtr = (uint8 *)DataPtr; - - for ( i = 0 ; i < DataLength ; i++, BufPtr++) - { - Index = ( ( Crc ^ *BufPtr) & 0x00FF); - Crc = ( (Crc >> 8 ) & 0x00FF) ^ CrcTable[Index]; - } - break; - -/* case CFE_ES_CRC_8: */ -/* CFE_ES_WriteToSysLog("CFE ES Calculate CRC8 not Implemented\n"); */ -/* break; */ - - default: - break; + /* case CFE_ES_CRC_32: */ + /* CFE_ES_WriteToSysLog("CFE ES Calculate CRC32 not Implemented\n"); */ + /* break; */ + + case CFE_ES_CRC_16: + Crc = (int16)(0xFFFF & InputCRC); + BufPtr = (uint8 *)DataPtr; + + for (i = 0; i < DataLength; i++, BufPtr++) + { + Index = ((Crc ^ *BufPtr) & 0x00FF); + Crc = ((Crc >> 8) & 0x00FF) ^ CrcTable[Index]; + } + break; + + /* case CFE_ES_CRC_8: */ + /* CFE_ES_WriteToSysLog("CFE ES Calculate CRC8 not Implemented\n"); */ + /* break; */ + + default: + break; } - return(Crc); + return (Crc); } /* End of CFE_ES_CalculateCRC() */ - - -int main( int argc, char **argv ) +int main(int argc, char **argv) { - int readSize; - int skipSize = 0; - int fileSize = 0; - uint32 fileCRC = 0; - int fd; - int done = 0; - char buffer[100]; - - /* check for valid input */ - if ( ( argc != 2 ) || ( strncmp(argv[1], "-help", 100) == 0 ) ) - { - printf("\ncFE TS CRC calculator for LRO files."); - printf("\nUsage: cfe_ts_crc [filename]\n"); + int readSize; + int skipSize = 0; + int fileSize = 0; + uint32 fileCRC = 0; + int fd; + int done = 0; + char buffer[100]; + + /* check for valid input */ + if ((argc != 2) || (strncmp(argv[1], "-help", 100) == 0)) + { + printf("\ncFE TS CRC calculator for LRO files."); + printf("\nUsage: cfe_ts_crc [filename]\n"); exit(0); } /* Set to skip the header (116 bytes) */ - skipSize = 116; - /* open the input file if possible */ - fd = open( argv[1], O_RDONLY ); - if ( fd < 0 ) - { - printf("\ncfe_ts_crc error: can't open input file!\n"); - exit(0); - } - /* seek past the number of bytes requested */ - lseek( fd, skipSize, SEEK_SET ); - - /* read the input file 100 bytes at a time */ - while ( done == 0 ) - { - readSize = read(fd, buffer, 100); - fileCRC = CFE_ES_CalculateCRC(buffer, readSize, fileCRC, CFE_ES_CRC_16); - fileSize += readSize; - if (readSize != 100) done=1; - } - /* print the size/CRC results */ - printf("\nTable File Name: %s\nTable Size: %d Bytes\nExpected TS Validation CRC: 0x%08X\n\n", argv[1], fileSize, fileCRC); + skipSize = 116; + /* open the input file if possible */ + fd = open(argv[1], O_RDONLY); + if (fd < 0) + { + printf("\ncfe_ts_crc error: can't open input file!\n"); + exit(0); + } + /* seek past the number of bytes requested */ + lseek(fd, skipSize, SEEK_SET); + + /* read the input file 100 bytes at a time */ + while (done == 0) + { + readSize = read(fd, buffer, 100); + fileCRC = CFE_ES_CalculateCRC(buffer, readSize, fileCRC, CFE_ES_CRC_16); + fileSize += readSize; + if (readSize != 100) + done = 1; + } + /* print the size/CRC results */ + printf("\nTable File Name: %s\nTable Size: %d Bytes\nExpected TS Validation CRC: " + "0x%08X\n\n", + argv[1], fileSize, fileCRC); /* Close file and check*/ - if( close(fd) != 0 ) + if (close(fd) != 0) { printf("\nerror: Cannot close file!\n"); exit(0); } - return(fileCRC); + return (fileCRC); }