hrev45868 adds 2 changesets to branch 'master' old head: 2aa0e54e4f600ae35bbb47169349ca582167ca10 new head: fa49409097fd2e5fc52950f1536d058147e13331 overview: http://cgit.haiku-os.org/haiku/log/?qt=range&q=fa49409+%5E2aa0e54 ---------------------------------------------------------------------------- 15be2c6: Fix called C++ object pointer is null Signed-off-by: Jérôme Duval <jerome.duval@xxxxxxxxx> fa49409: Use delete[] instead of delete Signed-off-by: Jérôme Duval <jerome.duval@xxxxxxxxx> [ Murai Takashi <tmurai01@xxxxxxxxx> ] ---------------------------------------------------------------------------- 2 files changed, 7 insertions(+), 5 deletions(-) src/add-ons/translators/icns/ICNSLoader.cpp | 8 ++++---- src/bin/bfs_tools/lib/Inode.cpp | 4 +++- ############################################################################ Commit: 15be2c601874b5ea8317151e99a293202851cc74 URL: http://cgit.haiku-os.org/haiku/commit/?id=15be2c6 Author: Murai Takashi <tmurai01@xxxxxxxxx> Date: Wed Jul 17 10:57:01 2013 UTC Committer: Jérôme Duval <jerome.duval@xxxxxxxxx> Commit-Date: Wed Jul 17 16:29:29 2013 UTC Fix called C++ object pointer is null Signed-off-by: Jérôme Duval <jerome.duval@xxxxxxxxx> ---------------------------------------------------------------------------- diff --git a/src/bin/bfs_tools/lib/Inode.cpp b/src/bin/bfs_tools/lib/Inode.cpp index f13b615..9f4634f 100644 --- a/src/bin/bfs_tools/lib/Inode.cpp +++ b/src/bin/bfs_tools/lib/Inode.cpp @@ -621,8 +621,10 @@ Inode::EmptyInode(Disk *disk, const char *name, int32 mode) } Inode *object = new (std::nothrow) Inode(disk, inode); - if (object == NULL) + if (object == NULL) { free(inode); + return NULL; + } object->AcquireBuffer(); // this must not be deleted anymore! ############################################################################ Revision: hrev45868 Commit: fa49409097fd2e5fc52950f1536d058147e13331 URL: http://cgit.haiku-os.org/haiku/commit/?id=fa49409 Author: Murai Takashi <tmurai01@xxxxxxxxx> Date: Wed Jul 17 10:14:29 2013 UTC Committer: Jérôme Duval <jerome.duval@xxxxxxxxx> Commit-Date: Wed Jul 17 16:31:28 2013 UTC Use delete[] instead of delete Signed-off-by: Jérôme Duval <jerome.duval@xxxxxxxxx> ---------------------------------------------------------------------------- diff --git a/src/add-ons/translators/icns/ICNSLoader.cpp b/src/add-ons/translators/icns/ICNSLoader.cpp index 49f27e9..ee25585 100644 --- a/src/add-ons/translators/icns/ICNSLoader.cpp +++ b/src/add-ons/translators/icns/ICNSLoader.cpp @@ -72,7 +72,7 @@ ICNSLoader::ICNSLoader(BPositionIO *stream) &fIconFamily); if (status != 0) { - delete icnsDataBuffer; + delete[] icnsDataBuffer; return; } @@ -98,9 +98,9 @@ ICNSLoader::ICNSLoader(BPositionIO *stream) fFormatList.SortItems(compareTypes); - delete icnsDataBuffer; + delete[] icnsDataBuffer; - fLoaded = true; + fLoaded = true; } @@ -177,7 +177,7 @@ ICNSLoader::GetIcon(BPositionIO *target, int index) } target->Write(rowBuff, iconImage.imageWidth * sizeof(uint32)); } - delete rowBuff; + delete[] rowBuff; icns_free_image(&iconImage); return B_OK;