Skip to content

Commit 4ba4cae

Browse files
Jean-Baptiste Queruandroid code review
authored andcommitted
Merge "Fix SkBitmap::fPixels not being locked correctly"
2 parents 0e0942c + 35ef567 commit 4ba4cae

File tree

1 file changed

+12
-2
lines changed

1 file changed

+12
-2
lines changed

core/jni/android/graphics/Bitmap.cpp

Lines changed: 12 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -298,8 +298,18 @@ static bool Bitmap_compress(JNIEnv* env, jobject clazz, SkBitmap* bitmap,
298298
}
299299

300300
bool success = false;
301-
SkWStream* strm = CreateJavaOutputStreamAdaptor(env, jstream, jstorage);
302-
if (NULL != strm) {
301+
if (NULL != bitmap) {
302+
SkAutoLockPixels alp(*bitmap);
303+
304+
if (NULL == bitmap->getPixels()) {
305+
return false;
306+
}
307+
308+
SkWStream* strm = CreateJavaOutputStreamAdaptor(env, jstream, jstorage);
309+
if (NULL == strm) {
310+
return false;
311+
}
312+
303313
SkImageEncoder* encoder = SkImageEncoder::Create(fm);
304314
if (NULL != encoder) {
305315
success = encoder->encodeStream(strm, *bitmap, quality);

0 commit comments

Comments
 (0)