IssueTracker

CAMERAX VERSION: 1.1.1, 1.2.0

CAMERA APPLICATION NAME AND VERSION: (Settings > Apps > (app name) > version) CameraXBasic demo 1.0.0

ANDROID OS BUILD NUMBER: (Settings > About > Build number):

SP1A.210812.016.A136U1UES2BVI1

Note that this has happened on all Android 12 variants we've tested

DEVICE NAME: Samsung A13, Samsung A32, Samsung A12 (Only with Android 12)

DESCRIPTION:

LIST ANY EXPERIMENTAL FEATURES: (As an example - @ExperimentalCamera2Interop) None

STEPS TO REPRODUCE: Using CameraXBasic Demo app:

1. Modify CameraXBasic demo app and add a Location to the Metadata object used for OutputFileOptions (within CameraFragment)
2. Open the app and accept permissions
3. Capture a photo

OBSERVED RESULTS: A corrupt JPEG file is created with an invalid EXIF header. The length of the EXIF section appears to be invalid, which prevents the photo from being viewable.

In the app, the photo preview has an all black screen. The thumbnail does not show up either. In the we see Glide failing to create a thumbnail and the app is unable to get the dimensions of the photo. Inspecting these files with a hex editor, indicates an invalid EXIF header that has the wrong length field. Manually overwriting the length to match the size before the IDR block (indicated by FFD8) will allow these photos to be viewable.

EXPECTED RESULTS: Photo should be viewable. Thumbnail should be created.

REPRODUCIBILITY: (5 of 5, 1 of 100, etc)

ADDITIONAL INFORMATION: We released this code in our app and have seen thousands of corrupted photos come through on several variants of Samsung A series devices running Android 12. This problem did not reproduce with the devices with the stock version of Android they came with. These devices are supposedly tested by google test lab but it seems they must not be testing them with Android 12.

CODE FRAGMENTS (this will help us troubleshoot your issues): See: https://github.com/mwolfeaxon/cameraxdemo This demo is the same as https://github.com/android/camera-samples/tree/master/CameraXBasic but with a Location object added to Metadata of OutputFileOptions.

Relevant code to reproduce:

  val metadata = Metadata().apply {
    isReversedHorizontal = lensFacing == CameraSelector.LENS_FACING_FRONT
    location = Location("fused").apply {
            latitude = 37.419857
            longitude = -122.0778827
            altitude = 0.0
            time = System.currentTimeMillis() - 1000
            accuracy = 13.747f
        }
    
  }
  val outputOptions = ImageCapture.OutputFileOptions.Builder(photoFile)
                        .setMetadata(metadata)
                        .build()
   imageCapture.takePicture(
                        outputOptions, cameraExecutor, object : ImageCapture.OnImageSavedCallback { ... }