Skip to content

Commit 14c0c98

Browse files
Jean-Baptiste QueruAndroid Git Automerger
authored andcommitted
am 4ba4cae: Merge "Fix SkBitmap::fPixels not being locked correctly"
* commit '4ba4caede125ff602b0d93f577f9054a07791ff7': Fix SkBitmap::fPixels not being locked correctly
2 parents 15e099c + 4ba4cae commit 14c0c98

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)