Fix memory leaks indicated by cppcheck #4
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
richiware
added a commit
that referenced
this pull request
Aug 3, 2018
9e7f35b Merge branch 'hotfix/enable_languages' into 'master' c4a3de2 Merge commit '5e926c428a5c97c9458cf69b0ba48db9ef2b23ab' into hotfix/enable_languages 562d44a Fixed error in cmake script with CXX language. a7a12f3 Merge branch 'feature/common_compile_opts' into 'master' dd2546f Hardening the warnings. 3f8331a Merge commit '1e27bfc7d3f03d0e0fd6a6f88a1f0a6cfba61377' as 'cmake/common' d91ddbd Removing wrong cmake/common. 268fe23 Fixing compilation errors on Windows. 4a0f1b4 Add size function. Added some const modifier. df48e52 Removed some inline modifiers c77a49f indent 4f659ab Removed unused images. d3a5b4a Removed old files. 890b118 Ref #3185. Improves sequences tests. aacc7b2 Ref #3185. Fixes PX4 warnings due to aligment conversion issues. Removes inline methods and change sequence API. Adds missing bracers. Adapts tests to new api. 2c57da5 Cleans unused makefiles 02cd30d Ref #3185. Fixes endianness checks issues when no endinanness check is done 04623b5 Fix bug in align_to function. 3dcb096 Added micro buffer remaining length function. d9d655d Added micro buffer length function utility. c8951b7 Refs #2884. Changed config.h.in acf934a Refs #2886. Change cmake configuration endian behavior. 2d929b3 Merge branch 'master' into develop 987e799 Refs #2883. Setup subtree for cmake macros. 2047be9 Adds C99 requirement. 0061c9a Refs #2870. Fixed shared libs bugs on Windows. 70ad72b Refs #2870. Fixed Windows warning. 12a1ea1 Refs #2866. Fixed Windows bugs. 766474a Refs #2866. Moved gtest.cmake to project folder. cedc95c Refs #2866. Added merger request advices. 4a795f7 Refs #2866. Removed string templates. 6d4361c Refs #2854. Fixed Windows CMake bugs. 4a14375 Refs #2838. Changed CMake tests architecture. 004cfbd Refs #2838. Checked installation on Linux. 4b40d88 Refs #2838. Checked installation on Windows. f339b30 Refs #2838. Remodeled CMake. 69fa53b Refs #2832. Changed versioning in CMake. 78d34ed Refs #2801. Changed templates to avoid warning on Windows. b0a2bfd Refs #2801. Compiling without warning on Windows. 4c02033 Refs #2810. Changed autogenerated header. cf29fa8 Refs #2773. Changed header string template. 7b0bea6 Fix endianness bug. ae461d6 Added init and reset function with offset. e756e38 Refs #2785. Remove jump functionality. b058623 Refs #2773. Modified string templates for MicroRTPSGen. 54aded8 Refs #2773. Added get_alignment functionality. 60d140f Minor change at reset buffer. 104096c Refs #2785. Added sizeof functions. Removed old examples. Added new example. d96bad7 Added new init_micro_buffer function for endian 1c23671 Refs #2662. Changes for accept merge request dca24be Refs #2662. Fix bug with the itetator position in arrays. All tests for sequences 92d2eba 2662 Added sequence functions. 01bc8b7 Refs #2662. Change api 84f0685 Refs #2662. Added funcionality (without test) 1c4c101 Refs #2583. Change examples and README format to markdown fdb15a2 Refs #2583. Change tests. 13c2574 Refs #2583. Removed resize funcionality ad1b9e2 Merge branch 'feature/windows' into 'develop' 1e1f10a Refs #2287. Fixed CMake Linux error. f91f259 Refs #2287. Fixed Windows test linking error. febd3ab Refs #2287. Fixed Windows compilation errors. 45e8c6a Refs:#2429, #2430. Adds returns and error flag. Adds tests. 9efc81d Merge branch 'develop' into feature/micrortpsgen 910c920 Refs #2391. Added new deallocate function to header and modified deserialization function. 5d95295 Merge branch 'develop' of git.sambaserver.eprosima.com:fastcdr/nanocdr into develop f3d78a1 Uses cmake old policy for version numbers 309c5ce Refs #2393. Adds missing line on deserialization c3eef39 Refs #2383. Added client.h to header file. fbecfae Refs #2381. .stg cleanning for c types f401638 Refs #2381. Avoid double \nin auto generated lines 9767808 Refs #2381. Adds simple IDL C types 6e65c09 Refs #2381. Added C Bacics Types. edcddfc Refs #2381. Change stg file from FastCdrCommon.stg to MicroCdrCommon.stg 922ea32 Creates branch for micrortpsgen related code cf0fd98 Refs #2375. Adds licenses afc70df Removes extern warning a4a754f Refs 2362. Extern Endianness check result e56a5cb Fixes Nuttx compilation. Fixeds endianness detection 04cc754 [fix bug]: serialization arrays with units with bytes 2,4,8 count their size bad e41b726 [fix bug]: alignment after arrays doesnt work fda82f0 MACHINNE_ENDIANNESS definition is public now e65c0db fix deserialize limit buffer bug 69f059f fix bug at array with elements higher than a byte 014b4ca fix endianness bug 5fe50b9 sequences removed 213736f fix endianness setup bbe0322 removed some code comments d8a95f6 added MicroState 1a34ba0 minor changes bdacfaf Remove examples/performance_cmp_versions 4a91c2a add to .gitignore 3986ff6 Actualizacion de README.rst 06ba6bd Release version 2.0.0 3e57dab Release version 2.0.0 e92a78e add basics tests b19d6a8 performance test. dynamic buffers. doc 8005b76 New API (definition and implementation) done (no tested). 334ad53 initial structure for the new changes 2ea6a1c initial structure for the new changes 4872048 Merge pull request #5 from dagar/pr-cmake_typo 54c5fec cmake fix message typo 50776f4 Merge pull request #4 from PeterDuerr/fix-memory-leaks 9b7fb94 Fix memory leaks indicated by cppcheck 74b2ed4 Code style: indirection operator attached to types 7cd4143 Code style: statement should be inside braces 98fd2cd Fixing code style 240a3cb Merge pull request #2 from dagar/pr-clang-tidy 8084fc6 readability add braces f219395 Refs #1960. Added new interface for static user buffers and its reuse 5cc9dcb Refs #1960. Add bool and bool array support bbca0a2 Refs #1960. Added extern 'C' to correct linkage in C++ projects 9eb3045 Refs #1960. Some changes to work with PX4 Firmware 7b2942c Refs #1960. Avoid double compilation for PX4/Firmware fdad641 Refs #1960. Some changes df2bf30 Merge pull request #1 from JavierIH/feature/refactor_micro e3555ac Refs #1960. replaced nano for micro 313ddf3 Refs #1960. fix cmakelists and added options for testing 2f8cec8 Refs #1960. Adding PX4 support 74454c5 Refs #1960. added changes to compile on stm32 6820b69 add table with nanoCDR implemented types ed3ab92 Add readme with minimal examples code f9baf1a fix typos on some comments 16f1c6e Update documentation c7224cc No memory leaks 5784178 repair memory leak 1b1159a Refs 1885 Buffer de lectura 0bc96f2 Refs 1885 Aligned and Non Aligned buffer ce51288 Refs 1885 nanoBuffer - nanoCDR bbf3551 Refs 1885 Alignment on write 4aa2738 Refs 1885 long to int64_t ea64d07 Refs 1885 Multithread 6c2cd6b Refs 1885 Change Unsigned long to uint64_t 0a4dc42 Refs 1885 Change Int to int32_t a608976 Refs 1875 Images to document acc383d Warnings StaticTest b73ab41 FAST to NANO 29ef95b Compile Windows 1eba3e9 Casting size_t to unisgned int 65e1b61 Casting size_t to unisgned int 492c43b Windows packaging f237bc7 Test endianness d996bd5 String Array and String Sequence 24325a9 GTest dynamic 4b6648f GTest dynamic and static 0b2d51c GTest static and dynamic bb4d1fb CMake conf b7a0de1 Document bdd7e5c Initial commit git-subtree-dir: cmake/common git-subtree-split: 9e7f35b10b011618c60ebc103970dda6ca22d636
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Running cppcheck, I noticed two little memory leaks...