Capture SDK: Crash fix related to bitmap (PM-150)

[obaidgini]

PM-150

[sonarqubecloud]

Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

[abolfazlimahdi]

Looks good. Thank you!

[Copilot]

Pull Request Overview

This PR fixes a crash related to bitmap handling in the PhotoThumbnail class by adding null safety checks to prevent NullPointerException when working with bitmap objects.

• Added null check for bitmap before accessing bitmap properties in rotation logic
• Improved null safety in the setBitmapOrBlack method by extracting bitmap reference

[Copilot]

This change introduces a potential issue where getRotatedBitmap() could return null (if the underlying bitmap is null), but the original logic assumed it would always return a valid bitmap when stackBitmap is not null. The null check should be performed after calling getRotatedBitmap() to handle cases where the method returns null.

[Copilot]

The null check for bitmap should be performed before calling bitmap.getWidth() and bitmap.getHeight() on line 139. If bitmap becomes null between this check and the createBitmap call, it will still cause a NullPointerException.

Suggested change

	Matrix matrix = new Matrix();
	matrix.postRotate(rotation);
	rotatedBitmap = Bitmap.createBitmap(bitmap, 0, 0, bitmap.getWidth(), bitmap.getHeight(), matrix, true);
	Bitmap localBitmap = bitmap; // Store a local reference to ensure consistency
	Matrix matrix = new Matrix();
	matrix.postRotate(rotation);
	rotatedBitmap = Bitmap.createBitmap(localBitmap, 0, 0, localBitmap.getWidth(), localBitmap.getHeight(), matrix, true);