Replacing sprintf with snprintf to avoid overflow.

.clang-format

@@ -0,0 +1,67 @@
+# Format Style Options - Created with Clang Power Tools
+---
+AccessModifierOffset: -4
+AlignAfterOpenBracket: Align
+AlignConsecutiveAssignments: true
+AlignConsecutiveDeclarations: true
+AlignConsecutiveMacros: true
+AlignEscapedNewlines: Left
+AlignOperands: true
+AlignTrailingComments: true
+AllowAllParametersOfDeclarationOnNextLine: true
+AllowShortBlocksOnASingleLine: Empty
+AllowShortCaseLabelsOnASingleLine: false
+AllowShortFunctionsOnASingleLine: Empty
+AllowShortIfStatementsOnASingleLine: false
+AllowShortLoopsOnASingleLine: false
+AlwaysBreakAfterReturnType: None
+AlwaysBreakBeforeMultilineStrings: false
+BinPackArguments: true
+BinPackParameters: true
+BreakBeforeBinaryOperators: None
+BreakBeforeBraces: Allman
+BreakBeforeTernaryOperators: true
+BreakStringLiterals: true
+ColumnLimit: 120
+CommentPragmas: ''
+ContinuationIndentWidth: 4
+Cpp11BracedListStyle: true
+DerivePointerAlignment: false
+ExperimentalAutoDetectBinPacking: false
+ForEachMacros: 
+  []
+IncludeBlocks: Preserve
+IncludeCategories: 
+  []
+IncludeIsMainRegex: $
+IndentCaseLabels: true
+IndentPPDirectives: None
+IndentWidth: 4
+KeepEmptyLinesAtTheStartOfBlocks: true
+Language: Cpp
+MacroBlockBegin: ''
+MacroBlockEnd: ''
+MaxEmptyLinesToKeep: 1
+PenaltyBreakAssignment: 2
+PenaltyBreakBeforeFirstCallParameter: 19
+PenaltyBreakComment: 300
+PenaltyBreakFirstLessLess: 120
+PenaltyBreakString: 1000
+PenaltyExcessCharacter: 1000000
+PenaltyReturnTypeOnItsOwnLine: 60
+PointerAlignment: Right
+ReflowComments: true
+SortIncludes: false
+SpaceAfterCStyleCast: false
+SpaceBeforeAssignmentOperators: true
+SpaceBeforeCpp11BracedList: true
+SpaceBeforeParens: ControlStatements
+SpaceInEmptyParentheses: false
+SpacesBeforeTrailingComments: 1
+SpacesInCStyleCastParentheses: false
+SpacesInParentheses: false
+SpacesInSquareBrackets: false
+Standard: c++11
+TabWidth: 8
+UseTab: Never
+...

elf2cfetbl.c

@@ -124,16 +124,16 @@ union Elf_Ehdr   ElfHeader;
 union Elf_Shdr **SectionHeaderPtrs                  = NULL;
 union Elf_Shdr   SectionHeaderStringTable           = {{0, 0, 0, 0, 0, 0, 0, 0, 0, 0}};
 int64            SectionHeaderStringTableDataOffset = 0;
-char **          SectionNamePtrs                    = NULL;
+char           **SectionNamePtrs                    = NULL;
 
 struct stat SrcFileStats;
 
 uint64_t          StringTableDataOffset = 0;
 int32             SymbolTableDataOffset = 0;
 uint64_t          NumSymbols            = 0;
 uint64_t          SymbolTableEntrySize  = 0;
-union Elf_Sym **  SymbolPtrs            = NULL;
-char **           SymbolNames;
+union Elf_Sym   **SymbolPtrs            = NULL;
+char            **SymbolNames;
 int32             TblDefSymbolIndex = -1;
 CFE_TBL_FileDef_t TblFileDef;
 int32             UserObjSymbolIndex = -1;
@@ -924,7 +924,7 @@ int32 ProcessCmdLineOptions(int ArgumentCount, char *Arguments[])
     bool      InputFileSpecified  = false;
     bool      OutputFileSpecified = false;
     int       i                   = 1;
-    char *    EndPtr;
+    char     *EndPtr;
     uint32    MaxDay;
     struct tm FileEpochTm;
     struct tm ScEpochTm;
@@ -2244,7 +2244,8 @@ int32 GetStringFromMap(char *Result, ElfStrMap *Map, int32 Key)
 
     if (Status == FAILED)
     {
-        sprintf(Result, Map->String, Key);
+        // Using snprintf to avoid overflow
+        snprintf(Result, sizeof(String), Map->String, Key);
     }
 
     return Status;