Android Camera takes multi lined bugged picture for some users

Question

Some users of my app are taking some bugged pictures that look like that:

The only thing I do is standard bitmap API in jpegCallback:

BitmapFactory.Options opts = new BitmapFactory.Options();
opts.inSampleSize = 4;
bm = BitmapFactory.decodeByteArray(data, 0, data.length, opts);
bm = Bitmap.createScaledBitmap(bm , 640, 480,  true);

and then write it on the disk

 imageFile = new File("/sdcard/app_dir/upload.jpg");
 FileOutputStream outStream = new FileOutputStream(imageFile);
 bm.compress(CompressFormat.JPEG, 75, outStream);
 outStream.flush();
 outStream.close();

edit: I've removed a call to setPreviewSize like explain there: Android: Jpeg saved from camera looks corrupted

I think it did help for some users (Desire HD), but I can tell others still got the issue (Desire S).

I really wish someone could explain the reason why pics looks distorded in the first place.

My Galaxy tab gets similar output when trying to Qik chat with other Galaxy tabs, so you're not the only one having this issue. — User1578, Jun 24 '11 at 14:19
similar bug with a possible solution http://stackoverflow.com/questions/5540981/picture-distorted-with-camera-and-getoptimalpreviewsize — plus-, Jun 24 '11 at 15:36
I had same issue with HTC Desire HD. Resolved by setting values `p.setPreviewSize(optimalPreviewSize.width, optimalPreviewSize.height);` and `p.setPictureSize(optimalPictureSize.width, optimalPictureSize.height);` — Martynas Jurkus, May 08 '13 at 06:27

score 1 · Answer 1 · answered Jul 14 '11 at 08:46

1

Well, it looks like you made some mistake with image size while decoding bitmap from byte array. Can you post code you are using for: - setting up camera - setting up decode parameters - retrieval of image data

answered Jul 14 '11 at 08:46

Konstantin Pribluda

12,329
1
30
35

score 1 · Accepted Answer · answered Jul 14 '11 at 18:47

I can't tell you why you're getting garbled data from some devices and not others, but I can suggest a workaround that seems to be working successfully for my app.

Your example code scales the camera's JPEG down to 640x480 before saving it off to the SD card. So I'm guessing you don't require the full-sized camera image.

If this assumption is true, you can skip Camera's takePicture() API entirely, and just save a preview frame to SD card. The easiest way to do this is with setOneShotPreviewCallback():

mCamera.setOneShotPreviewCallback( new StillPictureCallback() );

This will invoke once, and hand you back a buffer of data from the camera:

private class StillPictureCallback implements Camera.PreviewCallback {
    @Override
    public void onPreviewFrame(byte[] data, Camera camera) {
        mPictureTask = new SaveStillPictureTask();
        byte[] myData = null;
        if ( data != null ) {
            myData = data.clone();
        }
        mPictureTask.execute(myData);
    }
}

The callback invokes a background task to compress the data and save it to the file. The only bit of code I'm leaving out is the part that queries the camera for the preview frame format, width and height via getCameraInfo(). Note also that the Android YUVImage class was introduced with Froyo, so if you need to support earlier versions of Android, you will need to roll your own conversion code (there are examples here on StackOverflow).

/**
 * Background task to compress captured image data and save to JPEG file.
 * 
 */
private class SaveStillPictureTask extends AsyncTask<byte[], Void, Void> {

    private static final String TAG="VideoRecorder.SaveStillPictureTask";

    @Override
    protected Void doInBackground(byte[]... params) {
        byte[] data = params[0];
        FileOutputStream out = null;
        Bitmap bitmap = null;
        if ( data == null ) {
            Log.e(TAG, "doInBackground: data is null");
            return null;
        }

        try {
            out = new FileOutputStream(mSnapshotFilePath);

            // Use the preview image format, as documented in Android SDK javadoc
            if ( (mPreviewImageFormat == ImageFormat.NV21) || (mPreviewImageFormat == ImageFormat.YUY2) ) {
                saveYUVToJPEG( mCamera, out, data );
            } else if (mPreviewImageFormat == ImageFormat.JPEG) {
                Log.d(TAG, "directly write JPEG to storage");
                out.write(data);
            } else {
                Log.d(TAG, "try decoding to byte array");
                bitmap = BitmapFactory.decodeByteArray(data, 0, data.length);
                if ( bitmap != null ) {
                    bitmap.compress(Bitmap.CompressFormat.JPEG, 90, out);
                } else {
                    Log.e(TAG, "decodeByteArray failed, no decoded data");
                }
            }
        } 
        catch (FileNotFoundException ignore) {;} 
        catch (IOException ignore) {;}
        finally {
            if ( out != null ) {
                try {
                    out.close();
                } catch (IOException ignore) {;}
                out = null;
            }
            if ( bitmap != null ) {
                bitmap.recycle();
                bitmap = null;
            }
            data = null;
        }

        return null;
    }
}

/**
 * Save YUV image data (aka NV21 or YUV420sp) data to JPEG file.
 * 
 * @param camera
 * @param out
 * @param data
 */
protected void saveYUVToJPEG( Camera camera, FileOutputStream out, byte[] data ) {
    YuvImage yuvimg = null;
    try {
        int width = mPreviewWidth;
        int height = mPreviewHeight;

        Rect rect = new Rect();
        rect.left   = 0;
        rect.top    = 0;
        rect.right  = width  - 1;       
        rect.bottom = height - 1;       // The -1 is required, otherwise a buffer overrun occurs
        yuvimg = new YuvImage(data, mPreviewImageFormat, width, height, null);
        yuvimg.compressToJpeg(rect, 90, out);
    } finally {
        yuvimg = null;
    }
}

I'll give you the bounty as it ends tonight. While this is not the very answer I'm waiting for, I give you credits for teaching me about the onePreview callback. on a side note a friend of mine which is expert on those problematics say it is a common JPEG issue. I haven't had the time to work on that to confirm yet — plus-, Jul 14 '11 at 20:25
I think you mostly sidestep the JPEG issue with this approach. On nearly every device I've seen, you get the data as ImageFormat.NV21 (raw data) and then do JPEG encoding in your own code. — mportuesisf, Jul 14 '11 at 20:35
NV21 is default format and supported (must be) on every phone. Take care, that even is camera adverises certain preview size it does not mean that it is actually supported — Konstantin Pribluda, Jul 15 '11 at 07:24

score 1 · Answer 3 · answered Sep 02 '11 at 18:13

1

I had the exact same problem on an HTC Desire S.

I updated the mobile system per Settings -> About phone -> software updates

I also implemented the following code:

       Camera.Parameters parameters = mCamera.getParameters();
       parameters.setPictureFormat(PixelFormat.JPEG);
       mCamera.setParameters(parameters);

Worked for me.

answered Sep 02 '11 at 18:13

UncleIstvan

982
9
14

I already set PictureFormat to JPEG, and I can't see myself ask every Desire S user to update their phone. but thanks for the info – plus- Sep 03 '11 at 09:04

Android Camera takes multi lined bugged picture for some users

3 Answers3