[BugFix] Check status display

After you have edited the source code, post your patch here.
Forum rules
Home | Wiki | Rules

[BugFix] Check status display

Postby ivan386 » 31 Jan 2015 12:11

Attachments
Check status display.zip
(13.35 KiB) Downloaded 205 times
ivan386
 
Posts: 261
Joined: 17 Jun 2009 14:08

Re: [BugFix] Check status display

Postby raspopov » 01 Feb 2015 16:34

Не понял смысла, поясни.
(new BYTE [ 0 ] вобще-то допустимая инструкция, если что)
User avatar
raspopov
Project Admin
 
Posts: 945
Joined: 13 Jun 2009 12:30

Re: [BugFix] Check status display

Postby ivan386 » 01 Feb 2015 17:38

ivan386
 
Posts: 261
Joined: 17 Jun 2009 14:08

Re: [BugFix] Check status display

Postby raspopov » 01 Feb 2015 18:40

Это понятно, а зачем DownloadWithTorrent.cpp-то менять?
User avatar
raspopov
Project Admin
 
Posts: 945
Joined: 13 Jun 2009 12:30

Re: [BugFix] Check status display

Postby ivan386 » 01 Feb 2015 20:08

Так не будет иллюзии что есть хешсет торрента. Я то все думал откуда там мусор. Если нулевой хешсет задан спциально то оставь только DownloadWithTiger.cpp. Но мне кажется нулевой хешсет будет за зря память фрагментиовать и сбивать столку.
ivan386
 
Posts: 261
Joined: 17 Jun 2009 14:08

Re: [BugFix] Check status display

Postby raspopov » 02 Feb 2015 04:46

Если это править, то логично и в других местах тоже, и с TigerTree, и с ED2K хэшсетами тоже самое же...
User avatar
raspopov
Project Admin
 
Posts: 945
Joined: 13 Jun 2009 12:30

Re: [BugFix] Check status display

Postby ivan386 » 02 Feb 2015 05:45

У торрента выделение нулевого хешсета при btih в магните гарантированно происходит. У остальных насколько я понимаю только если нам подсунут нулевой хешсет а до этого сохраняется нулевой указатель.

В общем надо все места.

if ( m_p[Torrent|Tiger|Hashset]Block )

заменить на

if ( m_p[Torrent|Tiger|Hashset]Block && m_n[Torrent|Tiger|Hashset]Block )

либо макрос написать

Попробую сделать полностью.
ivan386
 
Posts: 261
Joined: 17 Jun 2009 14:08

Re: [BugFix] Check status display

Postby ivan386 » 02 Feb 2015 07:54

Attachments
hashsetblock pointer and size check.zip
(29.56 KiB) Downloaded 203 times
ivan386
 
Posts: 261
Joined: 17 Jun 2009 14:08

Re: [BugFix] Check status display

Postby raspopov » 09 Feb 2015 17:43

I hope that some time I will find a time for Shareaza to analyze this issue completely but now I committed a fix inspired by your suggestions: r9532.

ivan386, проверь, пожалуйста, что я всё правильно сделал.
User avatar
raspopov
Project Admin
 
Posts: 945
Joined: 13 Jun 2009 12:30


Return to Code Submission

Who is online

Users browsing this forum: No registered users and 1 guest

cron