Ping is so mean 🥺

AceQuorthon, 2 months ago

I got your IP adress, see you soon kiddo 😎

roguetrick, 2 months ago

I'm already in the network... Wait...

Bezier, 2 months ago

OP must be my neighbour

NotMelon, 2 months ago

Hahahahah you just doxed yourself hahahahah

Rooki, 2 months ago

Ahh yes get doxed through local ip XD

Thade780, 2 months ago

Woooooosh

felbane, 2 months ago

See, even ping hates “consistent device naming!”

bleistift2, 2 months ago

It’s actually a technical term in language theory – a term that cannot be produced by the language, but is still considered part of the ‘universe’ (i.e., the set of all terms) is called ‘garbage’ or ‘junk’.

Since I can’t find a source online to verify this claim, this might just have been the case in my courses…

damium, 2 months ago

“Invalid” or “unparseable” are more understandable descriptors in normal language. I don’t think I ever heard of garbage/junk being used for that in language theory but it may be domain specific usage.

popekingjoe, 2 months ago

Same tbh.

Sylver, 2 months ago

Wait a second… That’s MY ip!

herrcaptain, 2 months ago

Wait … the ping is coming from inside the house!

0x4E4F, 2 months ago

Holy shit 😱 (unplugs everything from mains… waits in terror in a corner…)

0x0, 2 months ago

Wow, that is an unhelpful error message. It could have told you it was expecting a number. It turns out that -i is short for “interval” and expects a number, whereas -I is used to specify an interface.

aniki, 2 months ago

This exactly. And because this is open-source, its super easy to track this down. I searched for the source of the error message:

github.com/iputils/iputils/blob/…/ping.c#L237

see this is ping_strtod function - so search for that function name and the first hit is the ‘-i’ case:

<span style="color:#323232;">		case 'i':
</span><span style="color:#323232;">		{
</span><span style="color:#323232;">			double optval;
</span><span style="color:#323232;">
</span><span style="color:#323232;">			optval = ping_strtod(optarg, _("bad timing interval"));
</span><span style="color:#323232;">			if (isless(optval, 0) || isgreater(optval, (double)INT_MAX / 1000))
</span><span style="color:#323232;">				error(2, 0, _("bad timing interval: %s"), optarg);
</span><span style="color:#323232;">			rts.interval = (int)(optval * 1000);
</span><span style="color:#323232;">			rts.opt_interval = 1;
</span><span style="color:#323232;">		}
</span><span style="color:#323232;">			break;
</span>

0xb, 2 months ago

-Me talking to people on twitter after nov 22 and before leaving

krnl386, 2 months ago

Good old git blame lol! Not only can you determine when the change was made and where, it’s trivial to look up the author of the commit: github.com/…/562e0d570d93cfcfdebab1215a2f04efa64a…

To be fair, the author’s first language may not be English…

Is anyone interested in submitting a pull request? Looks like Github contributions are accepted.

TheGingerNut, 2 months ago

My dude. It’s not a bug, OP just used small i instead of capital. Please don’t bother developers over this.

krnl386, 2 months ago

I think descriptive and useful error messages are OK to report as enhancements. They don’t have to be functional bugs.

TheGingerNut, 2 months ago

There might be a reason it was never fixed. Something stupid like a lot of 30 year old scripts relying on this out come. Hope not, you’re right that error message is ass