27097 2012-03-19 19:19:16 +0400 Не зануляется выделенное malloc-ом 2012-03-21 14:49:31 +0400 1 1 4 Development Sisyphus gcc4.5 unstable all Linux CLOSED NOTABUG P3 normal --- 21801 1 zerg sbolshakov glebfm sbolshakov sem timonbl4 qa-sisyphus oldest_to_newest 129889 0 zerg 2012-03-19 19:19:16 +0400 Как показал дисассемблер, при сборке под 32-битную систему, компилятор не правильно оптимизирует этот участок кода (org_herac_tuxguitar_player_impl_midiport_alsa_MidiSystem.c): JNIEXPORT jlong JNICALL Java_org_herac_tuxguitar_player_impl_midiport_alsa_MidiSystem_malloc(JNIEnv* env, jobject obj) { jlong ptr = 0; handle_t *handle = (handle_t *) malloc( sizeof(handle_t) ); handle->seq = NULL; handle->address = NULL; memcpy(&ptr, &handle, sizeof( handle )); return ptr; } Не зануляются handle->seq и handle->address Тимур предлагает вот такой патч, но локально для прогрраммы: http://git.altlinux.org/people/timonbl4/packages/?p=tuxguitar.git;a=commitdiff;h=0b9ef63575b28cf1b03d043a4904de944fb28ea1 Подробности в http://bugs.altlinux.org/21801 129890 1 ldv 2012-03-19 19:22:12 +0400 А почему gcc4.3 а не gcc4.5? 129910 2 zerg 2012-03-20 14:38:13 +0400 (В ответ на комментарий №1) > А почему gcc4.3 а не gcc4.5? Промахнулся 129911 3 ldv 2012-03-20 14:56:15 +0400 (In reply to comment #0) > Как показал дисассемблер, при сборке под 32-битную систему, компилятор не > правильно оптимизирует этот участок кода > (org_herac_tuxguitar_player_impl_midiport_alsa_MidiSystem.c): > JNIEXPORT jlong JNICALL > Java_org_herac_tuxguitar_player_impl_midiport_alsa_MidiSystem_malloc(JNIEnv* > env, jobject obj) > { > jlong ptr = 0; > > handle_t *handle = (handle_t *) malloc( sizeof(handle_t) ); > handle->seq = NULL; > handle->address = NULL; > > memcpy(&ptr, &handle, sizeof( handle )); > > return ptr; > } > > Не зануляются handle->seq и handle->address Не верю, чтобы компилятор это соптимизировал. Что вам показал дизассемблер? P.S. Почему они не используют calloc? 129912 4 ldv 2012-03-20 15:06:31 +0400 Это вообще должна была быть 1-строчная функция: return (jlong) calloc(1, sizeof(handle_t)); Терять время на изучение аномалий в чужом бредовом коде нет никакого желания. 129928 5 sem 2012-03-21 11:34:47 +0400 (В ответ на комментарий №4) > Это вообще должна была быть 1-строчная функция: > > return (jlong) calloc(1, sizeof(handle_t)); Неа. Там возвращается адрес локальной переменной. Что хотели сказать этим авторы неизвестно, читать этот код действительно нет желания. 129932 6 timonbl4 2012-03-21 12:26:07 +0400 (In reply to comment #4) > Это вообще должна была быть 1-строчная функция: > > return (jlong) calloc(1, sizeof(handle_t)); Так работать не будет на 32 битной системе, т.к. jlong всегда 64 битная 129939 7 ldv 2012-03-21 14:18:16 +0400 (In reply to comment #6) > (In reply to comment #4) > > Это вообще должна была быть 1-строчная функция: > > > > return (jlong) calloc(1, sizeof(handle_t)); > > Так работать не будет на 32 битной системе, т.к. jlong всегда 64 битная Очего же не будет, еще как будет. 129940 8 sem 2012-03-21 14:49:31 +0400 (В ответ на комментарий №5) > Неа. Там возвращается адрес локальной переменной. Не надо читать багзиллу по утрам. Сморозил чушь, забудьте.