ALT Linux Bugzilla – Full Text Bug Listing
| Summary: | Не зануляется выделенное malloc-ом | ||
|---|---|---|---|
| Product: | Sisyphus | Reporter: | Sergey V Turchin <zerg> |
| Component: | gcc4.5 | Assignee: | Sergey Bolshakov <sbolshakov> |
| Status: | CLOSED NOTABUG | QA Contact: | qa-sisyphus |
| Severity: | normal | ||
| Priority: | P3 | CC: | glebfm, sbolshakov, sem, timonbl4 |
| Version: | unstable | ||
| Hardware: | all | ||
| OS: | Linux | ||
| Bug Depends on: | |||
| Bug Blocks: | 21801 | ||
А почему gcc4.3 а не gcc4.5? (В ответ на комментарий №1)
> А почему gcc4.3 а не gcc4.5?
Промахнулся
(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? Это вообще должна была быть 1-строчная функция: return (jlong) calloc(1, sizeof(handle_t)); Терять время на изучение аномалий в чужом бредовом коде нет никакого желания. (В ответ на комментарий №4) > Это вообще должна была быть 1-строчная функция: > > return (jlong) calloc(1, sizeof(handle_t)); Неа. Там возвращается адрес локальной переменной. Что хотели сказать этим авторы неизвестно, читать этот код действительно нет желания. (In reply to comment #4) > Это вообще должна была быть 1-строчная функция: > > return (jlong) calloc(1, sizeof(handle_t)); Так работать не будет на 32 битной системе, т.к. jlong всегда 64 битная (In reply to comment #6) > (In reply to comment #4) > > Это вообще должна была быть 1-строчная функция: > > > > return (jlong) calloc(1, sizeof(handle_t)); > > Так работать не будет на 32 битной системе, т.к. jlong всегда 64 битная Очего же не будет, еще как будет. (В ответ на комментарий №5)
> Неа. Там возвращается адрес локальной переменной.
Не надо читать багзиллу по утрам. Сморозил чушь, забудьте.
|
Как показал дисассемблер, при сборке под 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