Merge "mksquashfsimage.sh: Make disable-4k-align a paramter" into nyc-dev

mksquashfsimage.sh: Make disable-4k-align a paramter

BUG: 29388879
Change-Id: I6ede2273419ba986d08decc0540741c54ed736b1
Signed-off-by: Mohamad Ayyash <mkayyash@google.com>

Merge changes from topic 'squashfs_ota_incrementals' into nyc-dev

* changes:
  Disable fragments and duplicates detection for squashfs
  mksquashfsimage.sh: Add support for passing block map files

Disable fragments and duplicates detection for squashfs

They don't play nicely with generating block maps. Fragments reduce
performance. A later patch will add workaround for duplicate detection.

BUG: 22322817
Change-Id: I01c731202ef7a2450b5d1beca627a0158c9cc3b4
Signed-off-by: Mohamad Ayyash <mkayyash@google.com>

mksquashfsimage.sh: Add support for passing block map files

BUG: 22322817
Change-Id: I7e17f6ad3a503ee97f7a81a1a8382cff47a913c6
Signed-off-by: Mohamad Ayyash <mkayyash@google.com>

Merge "simpleperf: restore perf_harden after simpleperf cts test." into nyc-dev

simpleperf: restore perf_harden after simpleperf cts test.

Bug: 29264324
Change-Id: Ia78ef924f049630968d222729cf3c1f378105b8a
(cherry picked from commit 5896132e00c7e49d2468d7ebd5372654df29b872)

simpleperf: check perf event limit.

The property security.perf_harden is added in https://android-review.googlesource.com/#/c/233736/5. And simpleperf needs to notice that.

Bug: 29054680
Change-Id: I5f1593f5b389d182a56c4bf3bd438a1dc2b66686
(cherry picked from commit ebf79f3a65c81ef0f8cd7a3b875771be88157fcc)

Do not build multinetwork tools when targeting the PDK

Bug: 29071416
Change-Id: Ie39cad3151907aabbf523895d6cde37b82ff29a2

libfec: return raw and corrected verity metadata signatures

Since it's not possible for libfec to identify whether the signature
field was successfully corrected, return both and allow the caller to
validate either signature.

Bug: 28943429
Change-Id: Ie913c21ba1d07d6df4c6feeb7226b2ec963f4d19

Add multinetwork debugging tools, dnschk and httpurl

Bug: 19537384
Bug: 27199751
Bug: 28719525
Change-Id: Ie983ec12ac6c550fa76c89cd44343220688a99b4

ANRdaemon: add README and a bash script to easily get trace. am: 9e90477a67
am: 4b7f0b27a2

* commit '4b7f0b27a267e5e10894bbf438dec15e534e4cc4':
  ANRdaemon: add README and a bash script to easily get trace.

Change-Id: I0ea20dd173cfaef70df4d07672810ca233fcb5f2

ANRdaemon: add a few new trace options and poke all userland services am: 5b42895d33
am: ecb5849a5d

* commit 'ecb5849a5d8cc59828c66910f2096ac44e455ca3':
  ANRdaemon: add a few new trace options and poke all userland services

Change-Id: Id35a4a5d9ccab90c0ab661e835fcd55787efa09a

ANRdaemon: increase the CPU threshold resolution to 0.01% am: 6d25c0bd81
am: c33bbcd4c7

* commit 'c33bbcd4c7a2700f28af68a8673bec71ec92aacc':
  ANRdaemon: increase the CPU threshold resolution to 0.01%

Change-Id: Ib6cc29b5ac94344c9c7a9766b6548563da060888

ANRdaemon: add README and a bash script to easily get trace.
am: 9e90477a67

* commit '9e90477a678b5b5fa6ac6dd7e78770746d6c50bf':
  ANRdaemon: add README and a bash script to easily get trace.

Change-Id: Ief3f17421cc970a6b8676921b414d093c43ea880

ANRdaemon: add a few new trace options and poke all userland services
am: 5b42895d33

* commit '5b42895d33d72e74e3e3446e1d43fc541fbf404f':
  ANRdaemon: add a few new trace options and poke all userland services

Change-Id: Ic1eac97f0b965cc8f1e2b73f26456975150acb48

ANRdaemon: increase the CPU threshold resolution to 0.01%
am: 6d25c0bd81

* commit '6d25c0bd81eee66810339d5db3d1ecd48691d229':
  ANRdaemon: increase the CPU threshold resolution to 0.01%

Change-Id: I4b8feac7f8908943ae543f096e97896e50963697

ANRdaemon: add README and a bash script to easily get trace.

Bug: 28989601
Change-Id: I5c9b819a6b05003e68d26bfacccf4c539ddab7ae

ANRdaemon: add a few new trace options and poke all userland services

Bug: 28989601
Change-Id: Ic0fdb6e0601d5e0a285d42cd9fe1a3d8d6f86286

ANRdaemon: increase the CPU threshold resolution to 0.01%

Bug: 28989601
Change-Id: If2a305bd14e8bd7c22115cbca668bd7777d0e7d2

Fail with an error if we can't read the policy for encryption

The absence of a policy reference in the unencrypted directory now
causes e4crypt_set_directory_policy to fail with an error. Callers
should call e4crypt_is_native (now moved into here) before calling this.

Bug: 28318405
Change-Id: I209292aba3abad3b19105c9afe2b84e8b3dd6874

Incr Ext4: Properly merge block_allocation lists

BUG: 27698960
Change-Id: Ia31b8319e0d1c8644ae1798116007a4b35c39e9b
Signed-off-by: Mohamad Ayyash <mkayyash@google.com>

Incremental Ext4 Bug: prevent out of bound boundary allocations

This happens around the boundary of block groups where a slightly
different block boundary is set based on size of filesystem which means
allocations based on a previous filesystem layout need to verify they're
not crossing that new bounday

BUG: 27698960
Change-Id: I45d444b4477f14f71e8f17144523505a7525b4e2
Signed-off-by: Mohamad Ayyash <mkayyash@google.com>

make_ext4fs: Modify ext4 filesystem setup to speed up fsck

1) Set EXT4_BG_INODE_UNINIT so fsck will skip scanning unused inodes
2) Use EXT4_FEATURE_RO_COMPAT_GDT_CSUM to enable group descriptor
checksums in filesystem which reduces overhead for fsck

BUG: 24747200
BUG: 28013112
Change-Id: I118ab26935442f452962b3a7c29c61acf76ad4d9
Signed-off-by: Mohamad Ayyash <mkayyash@google.com>

simpleperf: use libbacktrace_offline static library.

Device may not have libbacktrace_offline shared library, so always
using libbacktrace_offline static library.
Use simpleperf namespace to avoid conflicts with ThreadEntry in
libbacktrace.

Bug: 28152982
Change-Id: Ia2cdbee3b28556c790ccd7b061a801ed96f9b97c
(cherry picked from commit 040f7b4c62ce49f0af0746640e94d415764262cd)

Merge "Remove canned_fs_config from ext4_utils" into nyc-dev

Remove canned_fs_config from ext4_utils

It is now also used in mksquashfs so it was added to libcutils

BUG: 27467028
Change-Id: I81c87938d5b6b200a8ad817f0c64c5ca699f451c
Signed-off-by: Mohamad Ayyash <mkayyash@google.com>

Use default path for mmapPerf binaries

Change-Id: Ic6994988152942e162e0b977afe48144b4d8bb32

Convert mmapPerf to google-benchmark

Bug: 24272167
Change-Id: I831c9d26495e77e041a9461ddbafc70d7d11f9b7

Make squashfs work on BSD and macOS

Change-Id: Ie025a3d70237a7a959203d8cd066d05acf9e94b2

Use default path to put benchmark binaries

In order to keep all the benchmark binaries in the same
output place.

Change-Id: I188f7933c230d8955905217b915899dd19e614bf

Simpleperf: add test for reporting callgraph of shared libraries in apk file.

Also add comment for how to generate each perf.data in testdata/.
Add --log <leve> argument in unit test to help debugging.

Bug: 26962895
Change-Id: Iaa5a81cd8da5174d5b5b4e7847811e2432bf0db8
(cherry picked from commit 8f680f60dc800bec880c5c35bfbc1ac36165e1f3)

Merge "Update binder benchmark to use google-benchmark" into nyc-dev

Fix pagecache live stats.

Broke this while adding app filtering.

Change-Id: Ie9d4ae3b7f99169a15c04b9e2bc11d1cb2a62a95

Update binder benchmark to use google-benchmark

Bug: 26449138
Change-Id: Ie91d114bf669c686f618c252e58feff4c853ef91

Merge "Simpleperf: remove dependency on global current_arch." into nyc-dev

Simpleperf: remove dependency on global current_arch.

When running unit tests on arm64 devices:
[OK] ReportCommandTest.dwarf_callgraph
[OK] record_cmd.dwarf_callchain_sampling.
ERROR: can't unwind data recorded on a different architecture.

It is because ReportCommandtest.dwarf_callgraph opens a perf.data
recorded on x86_64, and changes current_arch. It causes a problem when
the test record_cmd.dwarf_callchain_sampling calls libbacktrace built
on aarch64. Athough it doesn't make the test fail, we should fix this.

Bug: 27860749

Change-Id: I2cd70369a769ef2199cab2302b8b824369be0907
(cherry picked from commit 48460892306fbc232d7623b1aa5be1aefdc1a8a7)

simpleperf: don't looking for _text symbol in /proc/kallsyms.

simpleperf looks for _text symbol in /proc/kallsyms to get
kernel address space information. But that info is no longer
needed. As some kernels don't have _text symbol, we should
remove the corresponding code.

Bug: 27814103
Change-Id: I4eb9869b58c51846c99b1204f304cd107a0eb354
(cherry picked from commit 7134f388ccdccb4a5e3025254002a85333a72a70)

aslr_test: Fix typo.

Bug: 26512380
Change-Id: Ia65986700f198a0f09db4a45a7d1bde173e557e6

Update simpleperf for LLVM rebase to r256229.

Bug: 26987366

(cherry picked from commit 07cdff03ffb5e69d73979e8c2f8585314751ee17)

Change-Id: I15f2c3ad2f3cbc6239f3ffe42dbc09da2b703d3a

Added -q flag to ignore errors.

dumpstate calls showmap for each pid, and since most of them are empty,
it ends up polluting logcat with entries like:

03-17 14:49:05.974 12160 12160 E dumpstate: command '/system/xbin/su
root showmap -q 9867' failed: No such file or directory

BUG: 26906985
Change-Id: I18d86adefe3f4b248f672732460d1145103e5828

Correct aslr_test minimum arm value.

Default mmap_rnd_bits value is now equal to the maximum supported (16)
on arm, not the minimum, 8.  Adjust the test criteria to match this.

Bug: 27680571
Change-Id: I16f2c3120c92ca9ea537f445ba24ebd8172d613f

Merge "Migrate aslr test to gtest." into nyc-dev

Merge "simpleperf: check if dwarf callgraph is supported by kernel before testing." into nyc-dev

Merge "Avoid bogus warnings by branching on whether dir is empty." into nyc-dev

simpleperf: check if dwarf callgraph is supported by kernel before testing.

Move inclusion of poll.h from header files to source files,
as event_selection_set.h is used by cmd_report_test.cpp, which
can be built on windows.

Bug: 27590546
Change-Id: Ic6b141a93d96610f911ab52da1d927b7f619ccb2
(cherry picked from commit 19e6b6d372a1244189dc2df62313085a61ed1a80)

Crypto performance benchmark

This reverts commit 39087e3740095e1851b0fcf1988c1c0881a208b6. And adds
the respective fix.

Change-Id: I1f70211c77f892c6252cf5e05e5e52d6c63e2293

Avoid bogus warnings by branching on whether dir is empty.

If the directory is empty, we try to set policy, otherwise we get it.

Bug: 27596572
Change-Id: I127534b7e43c95cf259f001b0acba255a117f76e

Merge "Revert "Crypto performance benchmark"" into nyc-dev

Revert "Crypto performance benchmark"

This reverts commit 00652d85b0228bf70905a4934e88a38471832ac2.

Change-Id: I550ebd2e25c275a03e4f93437e6f12f83e232c7f

Migrate aslr test to gtest.

Create tests which test the values of mmap_rnd_bits, and
mmap_rnd_compat_bits, if applicable, and verify that the address space
is randomized as expected given the provided values.

Also add a pair of tests to CTS that enforce that the observed entropy
is at least as high as a designated value.  That value will start as our
default value, which also corresponds to the maximum value of some
configurations.

Packaging of executables along with a nativetest suite is not supported,
so add a dummy nativetest suite, scrape_mmap_addr, which ensure that the
executables are present along with the given tests at predictable
locations.

Bug: 26512380
Change-Id: Ib1202984f9b98c503b8d3bc2c2248a9d06940845

Merge "Crypto performance benchmark" into nyc-dev

Crypto performance benchmark

Before running the benchmark do the following:

Lock the frequency of the core by setting scaling_min_freq=scaling_max_freq=DESIRED_FREQ

Change-Id: Ic291ae4db6a202ea2e968dc6823c61d352eb272d

Merge changes from topic 'incremental-ext4' into nyc-dev

* changes:
  make_ext4_fs: Clean up a few error messages
  Introduce blk_alloc_to_base_fs utility

cts: android.kernel.config

(cherry pick from commit 67522d05df61e7757e1ff08ba0b7ad0f19406ec0)

- Added plumbing for CTS tests
- Divide Required into CTS and gTest and
  Recommended into gTest
- Added check of CONFIG_ANDROID_LOGGER
- test typically completes after 3ms

Bug: 19173869
Change-Id: I13a92081bfa9b53abb69923b160b82e3be2142ab

su: Segmentation Fault on unknown numerical uids

(cherry pick from commit 45948f910c69d354cc3f7e81631445d83ebadc3d)

e.g.

su 2999,2999,2999 echo hello
Segmentation Fault

return from getpwuid is NULL. KISS we opt to unset LOGNAME and USER

Change-Id: I7e7eddacfd6857f83482727eb5ec5c51b9170039

cpustats: do not die in case a CPU is offlined

Change-Id: I05700266f1c7584ff51557b3e559654045379e87

Merge "Add support for filtering based on app name to pagecache." into nyc-dev

simpleperf: support building cts test.

1. build cts test libraries.
2. change tests to use tmpfile instead of perf.data.
3. support extracting testdata from cts test file.

Bug: 27387280
Change-Id: I7c5db77f3157d586d0c9beb446b247626e7cce36
(cherry picked from commit be7ec66eaa4f995bd9068637f7c7d5718173922c)

simpleperf: fix mac build.

I forgot that mac doesn't support whole static library. Change to static library
to avoid hiting the same problem in the future.

Bug: 26962895
Change-Id: Ia8e26c2e1cf7621391e00e100b22b7ed451dfa32
(cherry picked from commit ff7465c896ab90c162cbcf2b141c4cdd79bf437f)

simpleperf: port cmd_report_test to nonlinux.

And fix one build_id bug introduced by previous patch.

Bug: 26962895

Change-Id: Ibb8bd6ec77ee862bb01c26342d3b3024468e75b2
(cherry picked from commit 6e51bef9aa101307523946df301ca90215d0ee82)

simpleperf: fix build by removing off64_t.

Bug: 26962895
Change-Id: I211bacc22c2f6b6b24c639365c706e93d66ce4ac
(cherry picked from commit 41e32ca272f474d35a84a6e7c7f0a28c3160abc3)

simpleperf: report symbols of native libraries in apk file.

Changes included:
1. provide interface in read_apk.h to read build id and symbols.
2. report symbols of native libraries in apk file.
3. refactor code in read_elf.cpp and read_apk.cpp.
4. add verbose log.
5. add -o report_file_name option for report command.
6. add corresponding unit tests.

Bug: 26962895
Change-Id: I0d5398996e0c29dba4a6f5226692b758ca096bbd
(cherry picked from commit b1a885b014540a2f7798b5a35ea0f0ec150d93ee)

simpleperf: accept different perf_attr sizes.

The size of perf_attr can be increased as new features are added. And
we can accept different perf_attr sizes in perf.data, and ignore
unknown features.

Bug: 26962895

Change-Id: Ia42617f92acb3ab6ae56eb91aa0d7b73efc0b8af
(cherry picked from commit dd2ae2878b7e5ac1754da55a08cbcdacadf53d78)

simpleperf: port read_apk and read_apk/read_elf tests to nonlinux.

Also make following small changes:
1. Manage testdata files like libziparchive.
2. Replace StringToPid() with android::base::ParseInt().
3. Add tests in read_elf_test.

Bug: 26962895

Change-Id: I6603e9f3cea3a013e2618ea003790d102739f843
(cherry picked from commit 569f64afd3db2b43003978765b833669f99477d2)

make_ext4_fs: Clean up a few error messages

Change-Id: I269859a2c6beefb154fd483a02b29649416d60f9
Signed-off-by: Mohamad Ayyash <mkayyash@google.com>

Introduce blk_alloc_to_base_fs utility

Converts regular block mapping files into the new base_fs file format
for generating incremental ext4 images

Internal Design Doc: go/incremental-ext4
BUG: 26839493
Change-Id: If8dbb49718226771495c044eb1a8403e99a971fb
Signed-off-by: Mohamad Ayyash <mkayyash@google.com>

Add support for filtering based on app name to pagecache.

Change-Id: I571d254d69a255dd1b9fd7f33d98e7daa4b5ad5f

Fix pagemap_test#maps.

We don't necessarily have a brk-based heap.

Bug: http://b/27425642

(cherry picked from commit b2084a55a9e499b8c26b751956078282e12b0609)

Change-Id: Ia5c96fb26a8011c62037f3a3c4cbf44e3b936507

Don't define incremental ext4 for windows

Signed-off-by: Mohamad Ayyash <mkayyash@google.com>
Change-Id: I1eba14f7f4db2174de8feaa9d8e5e41524ac1d32

Redesign make_ext4fs to incrementally generate ext4 images

Allows passing a base fs mapping file through -d which preserves the
location of those mapping in existing files

Internal Design Doc: go/incremental-ext4
BUG: 26839493
Change-Id: I05e296693429d39466d257d1d0a3daf00510dc26
Signed-off-by: Mohamad Ayyash <mkayyash@google.com>

Revert "Redesign make_ext4fs to incrementally generate ext4 images"

This reverts commit 3d960843a2260a98c8df5cc39ca0dbf3d675f1a2.

Change-Id: I0fb7d65f094feb4013924685270fc847693b6889

Redesign make_ext4fs to incrementally generate ext4 images

Allows passing a blk mapping file through -Z which preserves the
location of those mapping in existing files

Internal Design Doc: go/incremental-ext4
BUG: 26839493
Change-Id: I447873bf65ae66dab9923e5a11c497f0722750df
Signed-off-by: Mohamad Ayyash <mkayyash@google.com>

rtc_time: test agent timeouts

(cherry pick from commit 9258225ece5d6816ddf9a45e45fb694142c32761)

Break the test up into four, keeping each well below the 60 second
timeout for the test agents. Overhead for the splitting up is an
additional 10 seconds in order to complete all four of the tests
instead of one. typical single test completion is 22.5 seconds.

Change-Id: If8e9a6c7dfd505a9fbc73e9248be4eb2b61521e5

Changes for mtp8996

Change-Id: I5fe5c365226087af47aaf7c5a576b7f2622f804d

Track update to bouncycastle 1.54

(cherry-picked from ab9a0c802d88ff47f3467fd66415a27f8f141b19)

Change-Id: I87252e179e191c6210d26e4055bee03fb43d8ba7

Merge "Add kernel config tests" into nyc-dev

Add kernel config tests

(cherry pick from commit 7129c7dcd20bf57cf3a64ec224e4ebac81c8cd88)

A simple start to simple day. Check for evidence of following CONFIGs:

CONFIG_MMC_BLOCK_MAX_SPEED=y
CONFIG_IPV6=y
CONFIG_IP_MULTICAST=y
CONFIG_PSTORE=y
CONFIG_PSTORE_CONSOLE=y
CONFIG_PSTORE_PMSG=y
CONFIG_SYSVIPC is not set

ToDo: Evidence for all configs in android/configs/android-base.cfg
      and android/configs/android-recommended.cfg. Most are not
      possible to directly discover, this is performed on a best
      effort basis.

Bug: 19173869
Bug: 26559308
Change-Id: I0447334fab7781579fab10610d2f7ac1d34e6044

Encrypt /data/misc

Now we have a workaround for 26989796 (we don't touch /data/misc/vold
thanks to:
    https://googleplex-android-review.git.corp.google.com/#/c/862867
) we can check this in and remove the need for a data wipe when we land
the proper kernel fix for this issue (stop caching negative file lookups
through a key change)

Bug: 26989796
Change-Id: I83e6b8395b2c6a607b692dc9cf71183b13680071

Merge "Add __s32 define." am: 9d63018815
am: 7991c03404

* commit '7991c034048ee5b376a16fb71f9f4b0a3354623d':
  Add __s32 define.

Merge "Add __s32 define."
am: 9d63018815

* commit '9d630188159868589992116dd5d67cecf17bf257':
  Add __s32 define.

Merge "Add __s32 define."

Add __s32 define.

This is needed for the new v4.4.1 kernel headers.

Bug: 23789423
Change-Id: I3053cdf895793c38ff1af41ebb560c13fda23215

Add methods to get and ensure encryption policy.

The ensure call will either assign an encryption policy, or verify
that an existing policy matches the expected value.  Uses the new
logging library so that customers can pivot logs into whatever
location they want: vold into logcat and init into dmesg.

Also add new directories that will have user-specific encryption
policy set on them so we avoid setting the default policy.

Bug: 25796509
Change-Id: Ia535630092822c80cde0939d8e46e6b47d9be2d8

Remove unencrypted properties

Change-Id: I59596be7c7df9151f4966a5051f06bcb087345dd

Merge "Support profiling of shared libs embedded in APKs." am: 4a5dafcc30
am: 615711f7ca

* commit '615711f7ca31518f72392d79d8a8aff3efb4ae34':
  Support profiling of shared libs embedded in APKs.

Merge "Support profiling of shared libs embedded in APKs."
am: 4a5dafcc30

* commit '4a5dafcc3093d55fc9d135b9a18c73bd96bcf3b1':
  Support profiling of shared libs embedded in APKs.

Merge "Support profiling of shared libs embedded in APKs."

Merge "Stop living in the \'80s." am: 1a5e41d3a1
am: 3778bcf5b3

* commit '3778bcf5b36fe0bd4f48a930fa0a68020e912803':
  Stop living in the '80s.

Merge "Stop living in the \'80s."
am: 1a5e41d3a1

* commit '1a5e41d3a1a7c1289503643d88748516649383ed':
  Stop living in the '80s.

Merge "Stop living in the '80s."

Stop living in the '80s.

Build fix for GCC.

Change-Id: I2215913647a95897dc39b862696e395a1ebcf208

Merge "Fix warnings in system/extras." am: 4eb09a62f2
am: 1b0f797a72

* commit '1b0f797a7274737468be7259ed9fde448b54af14':
  Fix warnings in system/extras.

Merge "Fix warnings in system/extras."
am: 4eb09a62f2

* commit '4eb09a62f2fc5f29747afd199dd1072d949b4892':
  Fix warnings in system/extras.

Merge "Fix warnings in system/extras."

Fix warnings in system/extras.

Bug: http://b/26936282
Change-Id: I1b9c6c9bb06944c32abcb871279d056eea0fb11a

Support profiling of shared libs embedded in APKs.

Some APKs contain shared libraries that the linker handles
by mmap'ing directly from their APKs (if the library is
uncompressed and the proper manifest flag is set). With
this patch simpleperf now breaks out samples on a per-li
basis and reports the name of the lib within the APK.
Example output:

  Cmdline: /system/xbin/simpleperf record -a sleep 30
  Samples: 140672 of event 'cpu-cycles'
  Event count: 84111474884

  Overhead  Command          Pid    Tid    Shared Object
  90.22%    b_open_from_apk  19066  19066
  /data/app/com.android.frameworks.coretests.install_jni_lib_open_from_apk-2/base.apk!lib/armeabi-v7a/libgcdstuff.so
  4.85%     b_open_from_apk  19066  19066
  /data/app/com.android.frameworks.coretests.install_jni_lib_open_from_apk-2/base.apk!lib/armeabi-v7a/libframeworks_coretests_jni.so
  1.19%     simpleperf       19085  19085  /system/lib/libc.so
  ...

Bug: 22560619
Change-Id: I1e0f2e155e03b33935eac24e104c3fd7b9a7e33c

Merge "Remove e4crypt_non_default_key"

Merge "Fix/suppress potential memory leaks warnings." am: dd681db9a7
am: 350c15c642

* commit '350c15c6422e80be1fd1da7cb5849c1de2aca51c':
  Fix/suppress potential memory leaks warnings.

Merge "Fix/suppress potential memory leaks warnings."
am: dd681db9a7

* commit 'dd681db9a7978abeb783af206406cfcae0b8514d':
  Fix/suppress potential memory leaks warnings.