Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]
Groups > linux.kernel > #1278547 > unrolled thread
| Started by | 김정배 <jb09.kim@samsung.com> |
|---|---|
| First post | 2015-11-27 06:50 +0100 |
| Last post | 2015-11-27 08:30 +0100 |
| Articles | 2 — 2 participants |
Back to article view | Back to linux.kernel
[PATCH] driver:dma bug_fix : access freed memory 김정배 <jb09.kim@samsung.com> - 2015-11-27 06:50 +0100
Re: [PATCH] driver:dma bug_fix : access freed memory Sudip Mukherjee <sudipm.mukherjee@gmail.com> - 2015-11-27 08:30 +0100
| From | 김정배 <jb09.kim@samsung.com> |
|---|---|
| Date | 2015-11-27 06:50 +0100 |
| Subject | [PATCH] driver:dma bug_fix : access freed memory |
| Message-ID | <qzk4V-78D-5@gated-at.bofh.it> |
RnJvbSA4ZjZhZWIzNjJkOWU0NGYyOWQ0NmFlNzY5NGNkZmVlNDQwODQwNmNlIE1vbiBTZXAgMTcg MDA6MDA6MDAgMjAwMQ0KRnJvbTogIktJTSBKVUdOQkFFIiA8amIwOS5raW1Ac2Ftc3VuZy5jb20+ DQpEYXRlOiBUaHUsIDI2IE5vdiAyMDE1IDE2OjI4OjQ3ICswOTAwDQpTdWJqZWN0OiBbUEFUQ0hd IGJ1Z19maXggOiBhY2Nlc3MgZnJlZWQgbWVtb3J5DQoNCnN5bmNfZmVuY19mcmVlICYgZmVuY2Vf Y2hlY2tfY2JfZnVuYyB3b3VsZCBiZSBleGVjdXRlZCBhdA0Kb3RoZXIgY3B1LiBmZW5jZV9jaGVj a19jYl9mdW5jIGFjY2VzcyBmcmVlZCBmZW5jZSBtZW1vcnkgYWZ0ZXINCmtmcmVlKGZlbmNlKSBh dCBzeW5jX2ZlbmNlX2ZyZWUuDQpUbyBlc2NhcGVkIHRoaXMgaXNzdWUsIGF0b21pY19yZWFkKCZm ZW5jZS0+c3RhdHVzKSBuZWVkIHRvIGJlDQpwcm90ZWN0ZWQgYnkgY2hpbGRfbGlzdF9sb2NrLg0K DQpTaWduZWQtb2ZmLWJ5OiAia2ltanVuZ2JhZVwiICIgPGpiMDkua2ltQHNhbXN1bmcuY29tPiIN Ci0tLQ0KIGRyaXZlcnMvZG1hLWJ1Zi9mZW5jZS5jICAgICAgICAgICAgICB8ICAgMTMgKysrKysr KysrKysrKw0KIGRyaXZlcnMvc3RhZ2luZy9hbmRyb2lkL3N5bmMuYyAgICAgICB8ICAgMTAgKysr KysrKy0tLQ0KIGRyaXZlcnMvc3RhZ2luZy9hbmRyb2lkL3N5bmNfZGVidWcuYyB8ICAgIDIgKysN CiBpbmNsdWRlL2xpbnV4L2ZlbmNlLmggICAgICAgICAgICAgICAgfCAgICAxICsNCiA0IGZpbGVz IGNoYW5nZWQsIDIzIGluc2VydGlvbnMoKyksIDMgZGVsZXRpb25zKC0pDQoNCmRpZmYgLS1naXQg YS9kcml2ZXJzL2RtYS1idWYvZmVuY2UuYyBiL2RyaXZlcnMvZG1hLWJ1Zi9mZW5jZS5jDQppbmRl eCA3YmI5ZDY1Li5kNDAyMTFlIDEwMDY0NA0KLS0tIGEvZHJpdmVycy9kbWEtYnVmL2ZlbmNlLmMN CisrKyBiL2RyaXZlcnMvZG1hLWJ1Zi9mZW5jZS5jDQpAQCAtMzEwLDYgKzMxMCwxOSBAQCBmZW5j ZV9yZW1vdmVfY2FsbGJhY2soc3RydWN0IGZlbmNlICpmZW5jZSwgc3RydWN0IGZlbmNlX2NiICpj YikNCiB9DQogRVhQT1JUX1NZTUJPTChmZW5jZV9yZW1vdmVfY2FsbGJhY2spOw0KIA0KK2Jvb2wN CitmZW5jZV9yZW1vdmVfY2FsbGJhY2tfbG9ja2VkKHN0cnVjdCBmZW5jZSAqZmVuY2UsIHN0cnVj dCBmZW5jZV9jYiAqY2IpDQorew0KKwlib29sIHJldDsNCisNCisJcmV0ID0gIWxpc3RfZW1wdHko JmNiLT5ub2RlKTsNCisJaWYgKHJldCkNCisJCWxpc3RfZGVsX2luaXQoJmNiLT5ub2RlKTsNCisN CisJcmV0dXJuIHJldDsNCit9DQorRVhQT1JUX1NZTUJPTChmZW5jZV9yZW1vdmVfY2FsbGJhY2tf bG9ja2VkKTsNCisNCiBzdHJ1Y3QgZGVmYXVsdF93YWl0X2NiIHsNCiAJc3RydWN0IGZlbmNlX2Ni IGJhc2U7DQogCXN0cnVjdCB0YXNrX3N0cnVjdCAqdGFzazsNCmRpZmYgLS1naXQgYS9kcml2ZXJz L3N0YWdpbmcvYW5kcm9pZC9zeW5jLmMgYi9kcml2ZXJzL3N0YWdpbmcvYW5kcm9pZC9zeW5jLmMN CmluZGV4IDgzZjY0MGYuLjliNjA1NDIgMTAwNjQ0DQotLS0gYS9kcml2ZXJzL3N0YWdpbmcvYW5k cm9pZC9zeW5jLmMNCisrKyBiL2RyaXZlcnMvc3RhZ2luZy9hbmRyb2lkL3N5bmMuYw0KQEAgLTUx OCwxMiArNTE4LDE2IEBAIHN0YXRpYyBjb25zdCBzdHJ1Y3QgZmVuY2Vfb3BzIGFuZHJvaWRfZmVu Y2Vfb3BzID0gew0KIHN0YXRpYyB2b2lkIHN5bmNfZmVuY2VfZnJlZShzdHJ1Y3Qga3JlZiAqa3Jl ZikNCiB7DQogCXN0cnVjdCBzeW5jX2ZlbmNlICpmZW5jZSA9IGNvbnRhaW5lcl9vZihrcmVmLCBz dHJ1Y3Qgc3luY19mZW5jZSwga3JlZik7DQotCWludCBpLCBzdGF0dXMgPSBhdG9taWNfcmVhZCgm ZmVuY2UtPnN0YXR1cyk7DQorCWludCBpOw0KKwl1bnNpZ25lZCBsb25nIGZsYWdzOw0KIA0KIAlm b3IgKGkgPSAwOyBpIDwgZmVuY2UtPm51bV9mZW5jZXM7ICsraSkgew0KLQkJaWYgKHN0YXR1cykN Ci0JCQlmZW5jZV9yZW1vdmVfY2FsbGJhY2soZmVuY2UtPmNic1tpXS5zeW5jX3B0LA0KKwkJc3Bp bl9sb2NrX2lycXNhdmUoZmVuY2UtPmNic1tpXS5zeW5jX3B0LT5sb2NrLCBmbGFncyk7DQorCQlp ZiAoYXRvbWljX3JlYWQoJmZlbmNlLT5zdGF0dXMpKQ0KKwkJCWZlbmNlX3JlbW92ZV9jYWxsYmFj a19sb2NrZWQoZmVuY2UtPmNic1tpXS5zeW5jX3B0LA0KIAkJCQkJICAgICAgJmZlbmNlLT5jYnNb aV0uY2IpOw0KKwkJc3Bpbl91bmxvY2tfaXJxcmVzdG9yZShmZW5jZS0+Y2JzW2ldLnN5bmNfcHQt PmxvY2ssIGZsYWdzKTsNCisNCiAJCWZlbmNlX3B1dChmZW5jZS0+Y2JzW2ldLnN5bmNfcHQpOw0K IAl9DQogDQpkaWZmIC0tZ2l0IGEvZHJpdmVycy9zdGFnaW5nL2FuZHJvaWQvc3luY19kZWJ1Zy5j IGIvZHJpdmVycy9zdGFnaW5nL2FuZHJvaWQvc3luY19kZWJ1Zy5jDQppbmRleCAwMTUzY2QyLi5l ODU0ZjM1IDEwMDY0NA0KLS0tIGEvZHJpdmVycy9zdGFnaW5nL2FuZHJvaWQvc3luY19kZWJ1Zy5j DQorKysgYi9kcml2ZXJzL3N0YWdpbmcvYW5kcm9pZC9zeW5jX2RlYnVnLmMNCkBAIC0xNTUsNyAr MTU1LDkgQEAgc3RhdGljIHZvaWQgc3luY19wcmludF9mZW5jZShzdHJ1Y3Qgc2VxX2ZpbGUgKnMs IHN0cnVjdCBzeW5jX2ZlbmNlICpmZW5jZSkNCiAJCQljb250YWluZXJfb2YoZmVuY2UtPmNic1tp XS5zeW5jX3B0LA0KIAkJCQkgICAgIHN0cnVjdCBzeW5jX3B0LCBiYXNlKTsNCiANCisJCXNwaW5f bG9ja19pcnFzYXZlKHB0LT5iYXNlLmxvY2ssIGZsYWdzKTsNCiAJCXN5bmNfcHJpbnRfcHQocywg cHQsIHRydWUpOw0KKwkJc3Bpbl91bmxvY2tfaXJxcmVzdG9yZShwdC0+YmFzZS5sb2NrLCBmbGFn cyk7DQogCX0NCiANCiAJc3Bpbl9sb2NrX2lycXNhdmUoJmZlbmNlLT53cS5sb2NrLCBmbGFncyk7 DQpkaWZmIC0tZ2l0IGEvaW5jbHVkZS9saW51eC9mZW5jZS5oIGIvaW5jbHVkZS9saW51eC9mZW5j ZS5oDQppbmRleCBkMTc0NTg1Li5hNWEzZmQxIDEwMDY0NA0KLS0tIGEvaW5jbHVkZS9saW51eC9m ZW5jZS5oDQorKysgYi9pbmNsdWRlL2xpbnV4L2ZlbmNlLmgNCkBAIC0yMjQsNiArMjI0LDcgQEAg c2lnbmVkIGxvbmcgZmVuY2VfZGVmYXVsdF93YWl0KHN0cnVjdCBmZW5jZSAqZmVuY2UsIGJvb2wg aW50ciwgc2lnbmVkIGxvbmcgdGltZW8NCiBpbnQgZmVuY2VfYWRkX2NhbGxiYWNrKHN0cnVjdCBm ZW5jZSAqZmVuY2UsIHN0cnVjdCBmZW5jZV9jYiAqY2IsDQogCQkgICAgICAgZmVuY2VfZnVuY190 IGZ1bmMpOw0KIGJvb2wgZmVuY2VfcmVtb3ZlX2NhbGxiYWNrKHN0cnVjdCBmZW5jZSAqZmVuY2Us IHN0cnVjdCBmZW5jZV9jYiAqY2IpOw0KK2Jvb2wgZmVuY2VfcmVtb3ZlX2NhbGxiYWNrX2xvY2tl ZChzdHJ1Y3QgZmVuY2UgKmZlbmNlLCBzdHJ1Y3QgZmVuY2VfY2IgKmNiKTsNCiB2b2lkIGZlbmNl X2VuYWJsZV9zd19zaWduYWxpbmcoc3RydWN0IGZlbmNlICpmZW5jZSk7DQogDQogLyoqDQotLSAN CjEuNy45LjUNCg0K -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
[toc] | [next] | [standalone]
| From | Sudip Mukherjee <sudipm.mukherjee@gmail.com> |
|---|---|
| Date | 2015-11-27 08:30 +0100 |
| Message-ID | <qzlDH-8cP-3@gated-at.bofh.it> |
| In reply to | #1278547 |
On Fri, Nov 27, 2015 at 05:44:53AM +0000, 김정배 wrote: > From 8f6aeb362d9e44f29d46ae7694cdfee4408406ce Mon Sep 17 00:00:00 2001 > From: "KIM JUGNBAE" <jb09.kim@samsung.com> > Date: Thu, 26 Nov 2015 16:28:47 +0900 > Subject: [PATCH] bug_fix : access freed memory This part should not be present in the patch. > > sync_fenc_free & fence_check_cb_func would be executed at > other cpu. fence_check_cb_func access freed fence memory after > kfree(fence) at sync_fence_free. > To escaped this issue, atomic_read(&fence->status) need to be > protected by child_list_lock. > > Signed-off-by: "kimjungbae\" " <jb09.kim@samsung.com>" The From name and the Signed-off-by name shouls be same. Mayvbe you can consider having Kim Jugnbae (or Jungbae) as both. > --- > drivers/dma-buf/fence.c | 13 +++++++++++++ > drivers/staging/android/sync.c | 10 +++++++--- > drivers/staging/android/sync_debug.c | 2 ++ > include/linux/fence.h | 1 + > 4 files changed, 23 insertions(+), 3 deletions(-) Usually staging patches can not touch anything outside staging. regards sudip -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
[toc] | [prev] | [standalone]
Back to top | Article view | linux.kernel
csiph-web