Bug#554620: cupt: Network problems not handled gracefully

2010-04-09 Thread James Vega
On Sat, Nov 7, 2009 at 6:18 PM, Eugene V. Lyubimkin
jackyf.de...@gmail.com wrote:
 James Vega wrote:
 Can you reproduce the issue again so we can debug it?

 That was the first and only time I've seen it so far.  The added checks
 look right, but I'm not sure I'll be able to verify whether it truly
 solves the problem.
 That checks hardly can solve the problem, they can post-handle it better. And
 for debugging purposes.

 I marked this bug as unreproducible. Let it dangle for a while.

 If you will ever reproduce this E: read from socket failed, I would like to
 see output of program with '-o debug::downloader=1' appended.

I just saw this again, and did not see any errors from Cupt.
Unfortunately, when I re-ran the update with debug::downloader=1 there
were no read failures.

-- 
James
GPG Key: 1024D/61326D40 2003-09-02 James Vega james...@debian.org



--
To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org



Bug#554620: cupt: Network problems not handled gracefully

2010-04-09 Thread Eugene V. Lyubimkin
package libcupt-perl
tags 554620 + pending
thanks

James Vega wrote:
 I just saw this again, and did not see any errors from Cupt.
 Unfortunately, when I re-ran the update with debug::downloader=1 there
 were no read failures.
 
Hi James, thanks for heads up. I had few similar failures since you filed that
bug, but also with no change to reproduce it - very rare. By symptoms it looks
like race condition, but since I use no threads in application (only forks and
Unix sockets)...

Nevertheless today I decided to have another look and with high probability I
found the reason finally - signal handler. Since the handler is being invoked
asynchronously, it can cause pseudo-race-condition situation.

So I wrote and committed a short patch, which, I believe, fixes this bug. If
you, me or someone else will encounter this bug with version having this
commit (af3577b) I will re-open the bug and look even more harder.

-- 
Eugene V. Lyubimkin aka JackYF, JID: jackyf.devel(maildog)gmail.com
C++/Perl developer, Debian Developer



signature.asc
Description: OpenPGP digital signature


Bug#554620: cupt: Network problems not handled gracefully

2009-11-07 Thread James Vega
On Thu, Nov 05, 2009 at 09:36:49PM +0200, Eugene V. Lyubimkin wrote:
  $ sudo cupt update
  Get:1 http://debian.lcs.mit.edu/debian squeeze Release
  Get:2 http://security.debian.org squeeze/updates Release
  0% [1 squeeze Release 0B][2 squeeze/updates Release 0B]
  
   | 0B/s | ETA: 0sE: read from socket failed:
 This is an error from Unix local socket, not network one, but I can't guess
 why it happened. Looks like something went bad with the listener socket, and
 some client sockets received error from read(), and some even got wrong data
 (which led to a crash).
 
 I just added two more internal checks to master branch though just in case.
 
 Can you reproduce the issue again so we can debug it?

That was the first and only time I've seen it so far.  The added checks
look right, but I'm not sure I'll be able to verify whether it truly
solves the problem.

-- 
James
GPG Key: 1024D/61326D40 2003-09-02 James Vega james...@debian.org


signature.asc
Description: Digital signature


Bug#554620: cupt: Network problems not handled gracefully

2009-11-07 Thread Eugene V. Lyubimkin
package cupt libcupt-perl
reassign 554620 libcupt-perl
tags 554620 + unreproducible
thanks

James Vega wrote:
 Can you reproduce the issue again so we can debug it?
 
 That was the first and only time I've seen it so far.  The added checks
 look right, but I'm not sure I'll be able to verify whether it truly
 solves the problem.
That checks hardly can solve the problem, they can post-handle it better. And
for debugging purposes.

I marked this bug as unreproducible. Let it dangle for a while.

If you will ever reproduce this E: read from socket failed, I would like to
see output of program with '-o debug::downloader=1' appended.

-- 
Eugene V. Lyubimkin aka JackYF, JID: jackyf.devel(maildog)gmail.com
C++/Perl developer, Debian Developer



signature.asc
Description: OpenPGP digital signature


Bug#554620: cupt: Network problems not handled gracefully

2009-11-05 Thread James Vega
Package: cupt
Version: 1.1.1
Severity: minor

Attempted to update and saw the following backtrace.  Looks like there
was an intermittent network problem, which caused some variable not
to be set.

$ sudo cupt update
Get:1 http://debian.lcs.mit.edu/debian squeeze Release
Get:2 http://security.debian.org squeeze/updates Release
0% [1 squeeze Release 0B][2 squeeze/updates Release 0B]

 | 0B/s | ETA: 0sE: read from socket failed:
E: error performing command 'update'
Use of uninitialized value $filename in hash element at
/usr/share/perl5/Cupt/Download/Manager.pm line 542.
 at /usr/share/perl5/Cupt/Download/Manager.pm line 542

Cupt::Download::Manager::download('Cupt::Download::Manager=HASH(0x8ec0168)',
'HASH(0x8f07fa0)') called at /usr/share/perl5/Cupt/System/Worker.pm
line 1582
eval {...} called at /usr/share/perl5/Cupt/System/Worker.pm line 1561

Cupt::System::Worker::update_release_and_index_data('Cupt::System::Worker=HASH(0x8860968)',
'Cupt::Download::Progresses::Console=HASH(0x8947238)') called at
/usr/bin/cupt line 1690
main::update_release_data() called at /usr/bin/cupt line 141
eval {...} called at /usr/bin/cupt line 140
main::main() called at /usr/bin/cupt line 155
 at /usr/share/perl5/Cupt/Core.pm line 105
Cupt::Core::mycatch() called at
/usr/share/perl5/Cupt/System/Worker.pm line 1775

Cupt::System::Worker::update_release_and_index_data('Cupt::System::Worker=HASH(0x8860968)',
'Cupt::Download::Progresses::Console=HASH(0x8947238)') called at
/usr/bin/cupt line 1690
main::update_release_data() called at /usr/bin/cupt line 141
eval {...} called at /usr/bin/cupt line 140
main::main() called at /usr/bin/cupt line 155
 at /usr/share/perl5/Cupt/Core.pm line 105
Cupt::Core::mycatch() called at /usr/bin/cupt line 143
main::main() called at /usr/bin/cupt line 155

-- System Information:
Debian Release: squeeze/sid
  APT prefers testing
  APT policy: (500, 'testing')
Architecture: i386 (i686)

Kernel: Linux 2.6.30-2-686 (SMP w/1 CPU core)
Locale: LANG=en_US.UTF-8, LC_CTYPE=en_US.UTF-8 (charmap=UTF-8)
Shell: /bin/sh linked to /bin/dash

Versions of packages cupt depends on:
ii  libcupt-perl  1.1.1 alternative front-end for dpkg --
ii  perl  5.10.1-5  Larry Wall's Practical Extraction
ii  sensible-utils0.0.1 Utilities for sensible alternative

cupt recommends no packages.

Versions of packages cupt suggests:
pn  libterm-readline-gnu-perl none(no description available)

-- no debconf information

-- 
James
GPG Key: 1024D/61326D40 2003-09-02 James Vega james...@debian.org



-- 
To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org



Bug#554620: cupt: Network problems not handled gracefully

2009-11-05 Thread Eugene V. Lyubimkin
package cupt libcupt-perl
reassign 554620 libcupt-perl
tags 554620 + unreproducible
thanks

James Vega wrote:
 Package: cupt
 Version: 1.1.1
 Severity: minor
 
 Attempted to update and saw the following backtrace.  Looks like there
 was an intermittent network problem, which caused some variable not
 to be set.

Hi James,
 $ sudo cupt update
 Get:1 http://debian.lcs.mit.edu/debian squeeze Release
 Get:2 http://security.debian.org squeeze/updates Release
 0% [1 squeeze Release 0B][2 squeeze/updates Release 0B]
 
  | 0B/s | ETA: 0sE: read from socket failed:
This is an error from Unix local socket, not network one, but I can't guess
why it happened. Looks like something went bad with the listener socket, and
some client sockets received error from read(), and some even got wrong data
(which led to a crash).

I just added two more internal checks to master branch though just in case.

Can you reproduce the issue again so we can debug it?

-- 
Eugene V. Lyubimkin aka JackYF, JID: jackyf.devel(maildog)gmail.com
C++/Perl developer, Debian Developer



signature.asc
Description: OpenPGP digital signature