Dear Rogier and Robert,

On Oct 03 2013, Rogier Wolff wrote:
> and it shows in the strace:
> getgid32()                              = 0
> setgid32(0)                             = 0
> getuid32()                              = 0
> setuid32(0)                             = 0
> geteuid32()                             = 0
> getuid32()                              = 0
> getegid32()                             = 0
> getgid32()                              = 0
> 
> which works as designed.... 

Yes, knowing what the code was supposed to do. :)

> 
> And then the code does: 
> 
>   /* reset the random seed */
>   srand (getpid());
> 
> time(NULL)                              = 1380776131
> 
> which should show up as a call to getpid, and not to "time". 
>    srand (time (NULL)); 
> 
> is a very common call to initialize the random generator, but arguably
> the getpid is better for mtr than the time variant.

It is a common method, indeed. I don't know how/why mtr uses a pseudo-random
generator, though, without having read the code (will read that later, if I
still have sufficient interest).

> If that works, it's a bug that has been fixed in the main codebase:
> either work around it by supplying a hostname, or upgrade if you want.

Yes, passing an argument works fine. Perhaps we need the newest version of
mtr packaged in Debian...

> (I personally think the default of "localhost" is untidy: cleanlyness
> would require mtr to say: "no host to trace provided. Bye!" (or just
> exit wihtout doing anything as it does in your case). However that
> means that starting mtr from a menu in a gui without arguments would
> make it close immediately.)

I don't really think that it is *that* unclean of a solution. OTOH, just
quitting without any message violates the spirit of Unix of being noisy when
errors occur, while being silent when everything is fine.

If mtr issued a message that I missed an argument, then I would not have
sent this bugreport in the first place. :)

Oh, I have a feature request: can you put a button to toggle the name
resolution in the GUI. :) This way, if name resolution is getting in the
way, then we can turn it off (or on) very easily.


Thanks a lot for mtr,

-- 
Rogério Brito : rbrito@{ime.usp.br,gmail.com} : GPG key 4096R/BCFCAAAA
http://cynic.cc/blog/ : github.com/rbrito : profiles.google.com/rbrito
DebianQA: http://qa.debian.org/developer.php?login=rbrito%40ime.usp.br


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

Reply via email to