Bug 7555 - Редактор id3v2 вставляет лишние символы в поле "Год"
Summary: Редактор id3v2 вставляет лишние символы в поле "Год"
Status: CLOSED FIXED
Alias: None
Product: Sisyphus
Classification: Development
Component: xmms (show other bugs)
Version: unstable
Hardware: all Linux
: P2 normal
Assignee: Michael Shigorin
QA Contact: qa-sisyphus
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2005-08-04 02:21 MSD by Eugene Vlasov
Modified: 2005-08-17 22:41 MSD (History)
2 users (show)

See Also:


Attachments
Обновленный патч yonas-id3v2 (105.73 KB, patch)
2005-08-06 14:47 MSD, Eugene Vlasov
no flags Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Eugene Vlasov 2005-08-04 02:21:51 MSD
eugene ~ $ rpmquery xmms    
xmms-1.2.10-alt7

Похоже на то, что при редактировании поля "Год" в id3v2 в фрейм TYER
прописываются лишние символы, что может смущать некоторые плэйеры.
До редактирования:

eugene ~ $ hexdump -C 01._I\'m_a_Fool_to_Want_You_\(Edited_Master_cs_8048\).mp3
| grep TYER -C 3                      
00000040  34 38 29 54 50 45 31 00  00 00 0f 00 00 00 42 69  |48)TPE1.......Bi|
00000050  6c 6c 69 65 20 48 6f 6c  69 64 61 79 54 41 4c 42  |llie HolidayTALB|
00000060  00 00 00 0e 00 00 00 4c  61 64 79 20 69 6e 20 53  |.......Lady in S|
00000070  61 74 69 6e 54 59 45 52  00 00 00 05 00 00 00 31  |atinTYER.......1|
00000080  39 35 38 43 4f 4d 4d 00  00 00 1e 00 00 00 58 58  |958COMM.......XX|
00000090  58 00 4d 50 33 20 63 6f  64 69 6e 67 20 62 79 20  |X.MP3 coding by |
000000a0  6e 65 72 76 61 73 79 73  74 65 6d 54 43 4f 4e 00  |nervasystemTCON.|

После редактирования тэгов в xmms:

eugene ~ $ hexdump -C 01._I\'m_a_Fool_to_Want_You_\(Edited_Master_cs_8048\).mp3
| grep TYER -C 3
00000070  65 72 20 63 73 20 38 30  34 38 29 54 50 45 31 00  |er cs 8048)TPE1.|
00000080  00 00 0f 00 00 00 42 69  6c 6c 69 65 20 48 6f 6c  |......Billie Hol|
00000090  69 64 61 79 54 41 4c 42  00 00 00 0e 00 00 00 4c  |idayTALB.......L|
000000a0  61 64 79 20 69 6e 20 53  61 74 69 6e 54 59 45 52  |ady in SatinTYER|
000000b0  00 00 00 1f 00 00 00 31  39 35 38 3f 4d 50 33 20  |.......1958?MP3 |
000000c0  63 6f 64 69 6e 67 20 62  79 20 6e 65 72 76 61 73  |coding by nervas|
000000d0  79 73 74 65 6d 43 4f 4d  4d 00 00 00 1e 00 00 00  |ystemCOMM.......|

Здесь видно, что длина фрейма TYER увеличилась до 1f, в нем после собственно
года идут остатки поля COMM (комментарий), а сам комментарий сдвинулся.
Comment 1 Michael Shigorin 2005-08-04 11:01:02 MSD
а сможете попробовать поправить?
Comment 2 Eugene Vlasov 2005-08-04 13:15:22 MSD
Попробовать могу, не могу обещать что руки скоро дойдут
Comment 3 Michael Shigorin 2005-08-04 13:24:19 MSD
ну, я-то об это вообще не спотыкался. :-)
Comment 4 Eugene Vlasov 2005-08-04 13:34:51 MSD
А я под это дело даже eyeD3 в свое время захакал (она год из таких тэгов читать
не могла).
Comment 5 Eugene Vlasov 2005-08-06 14:47:30 MSD
Created attachment 1030 [details]
Обновленный патч yonas-id3v2

Кажись зафиксил. Оказалось несложно, помогло добавление всего одной строчки -
обнуление записи v2_tag перед чтением полей формы.
Comment 6 Michael Shigorin 2005-08-06 21:15:14 MSD
Будьте добры, сделайте NMU xmms... и вообще -- прошу в команду :-)
Буду действительно признателен за помощь.
Comment 7 Eugene Vlasov 2005-08-06 21:48:41 MSD
Мне для NMU нужно что-то делать (писать на incoming@?). Ни разу не делал.
А для вступления в команду?
Comment 8 Eugene Vlasov 2005-08-17 20:03:47 MSD
Наконец то alt8 добралась до сизифа. Сейчас установил, проверил - вроде все в
порядке.
Comment 9 Michael Shigorin 2005-08-17 22:41:00 MSD
Вот и аюшки; спасибо Вам.