The Weird and The Wonderful

R. Giskard Reventlov10-Aug-12 12:33

R. Giskard Reventlov

10-Aug-12 12:33

Confirmation.asp?order=179000[^]

Lost interest after this. Smile | :)

"If you think it's expensive to hire a professional to do the job, wait until you hire an amateur." Red Adair.
nils illegitimus carborundum

me, me, me

Brisingr Aerowing10-Aug-12 14:13

Brisingr Aerowing

10-Aug-12 14:13

Yikes!

Bill Gates is a very rich man today... and do you want to know why? The answer is one word: versions.
Dave Barry
Read more at BrainyQuote[^]

krumia13-Aug-12 0:03

krumia

13-Aug-12 0:03

Okay what's wrong with that? It uses HTTPS and that should be good enough for everybody. Poke tongue | ;-P

Andrei Straut13-Aug-12 4:57

Andrei Straut

13-Aug-12 4:57

So were 512k some decades ago. Laugh | :laugh:

Also, look 2 threads below (incidentally, one that I've started). It mentions exactly this problem.

Full-fledged Java/.NET lover, full-fledged PHP hater.
Full-fledged Google/Microsoft lover, full-fledged Apple hater.
Full-fledged Skype lover, full-fledged YM hater.

BobJanova13-Aug-12 2:16

BobJanova

13-Aug-12 2:16

enhzflep13-Aug-12 4:46

13-Aug-12 4:46

That's gotta be the best laugh I've had all week. Since I was on a bender, thought I'd send this email to the company. Let their response time be a testament to how seriously they take security. Laugh | :laugh:

Gday Sir/Madam,

Have just read a forum post that lambastes your website for it's poor security. There are at least 2 problems with it as it stands

1) You've used a HTTP GET to pass variables to this page (the order number is present in the URL)
2) You've not authenticated the viewer as being the customer that placed the order.

For instance, I can enter the URL "https://www.alcatraztrips.com/Confirmation.asp?order=17900" and straight away see that Mary Cruz did attend the tour on 27 Sep 2005, leaving from Pier 33 at 11.15am


I can then enter the URL "https://www.alcatraztrips.com/Confirmation.asp?order=169000" and similarly I can see that Silvia Bollati is scheduled to attend a tour on the 25th August 2012 (13 days from now) Also departing pier 33, this time at 10am.


What if I or somebody else wanted to harm Silvia? Simple, run a program to harvest all the orders on your website, scan through them for the name of a purchaser of interest


It certainly doesn't take somebody that's particularly bright to understand that
(a) This is a massive security hole
(b) If somebody scheduled to attend the tour was located as a result of the service and subsequently murdered, your company would be held liable!!!

Kind of ironic for a website that deals in tours to a decommissioned Prison, don't you think?

You can view the lambasting here: http://www.codeproject.com/Messages/4335687/Alcatraz-the-tourist-website.aspx

Cheers,
Simon.

Make it work. Then do it better - Andrei Straut

Andrei Straut13-Aug-12 4:59

Andrei Straut

13-Aug-12 4:59

That was funny Laugh | :laugh:

Also, I've seen myself quoted in your sig (can't 5 twice), and although I'm flattered, you should know that it's not my invention, I've heard it on the interwebz somewhere Smile | :)

Full-fledged Java/.NET lover, full-fledged PHP hater.
Full-fledged Google/Microsoft lover, full-fledged Apple hater.
Full-fledged Skype lover, full-fledged YM hater.

enhzflep13-Aug-12 5:14

13-Aug-12 5:14

$Shucks | :-\$ Hope their customers data isn't left out on display for all to see. Also hoped someone else may get a giggle.

I'd be happy to attribute it to anon if you'd prefer.

I've found your posts to be both intelligent and helpful - the quote is also a good maxim - just one I'd never been clever enough to condense.

It's how things are often done - a good example is one that uses per-pixel manipulation. At first, it's a million times easier to just use SetPixel/GetPixel - with time after the algo is working one will often alter such ungainly access to something much harder to read and similarly quicker to execute.

I saw the quote in a post of yours and just HAD to steal it - (unquoted, hence the attributation to you) Smile | :)

Make it work. Then do it better - Andrei Straut

Andrei Straut13-Aug-12 5:51

Andrei Straut

13-Aug-12 5:51

Well, what can I say, thanks!

Full-fledged Java/.NET lover, full-fledged PHP hater.
Full-fledged Google/Microsoft lover, full-fledged Apple hater.
Full-fledged Skype lover, full-fledged YM hater.

Thou mewling ill-breeding pignut!

AspDotNetDev13-Aug-12 5:50

AspDotNetDev

13-Aug-12 5:50

Dear Sir/Madam,

It has recently been brought to our attention that your IP address has been viewing orders placed by customers with different IP addresses. This is a federal crime and you will be prosecuted if you do not cease and desist.

Sincerely,
Alcatraz Tours Security Official

enhzflep13-Aug-12 6:12

13-Aug-12 6:12

Nice

I'll see your taunt and raise you another..

Dear Mr Agent,

As with all police issued notices I receive reporting to be from the US, I will again take the time to remind you that I'm 12,650 kms away - that's 7,860 miles to the metrically challenged.

You've obviously been told by fellow ~~scammers~~ colleagues that you'l get better response rates when you claim to be US law enforcement than you'd enjoy with the old "We are related, I just need to use your bank account to wire $50,000,000 out of this country, you will receive a generous 5%"

Think I'll file this with my IRS note, my LAPD issued traffic infringement and my New York issued demand to attend the courthouse.

If only you realized I've never had a passpport, let alone left this country.

Why don't you head back into the capital city, Abuja? I hear there are many opportunities for those willing to work for them.. Laugh | :laugh:

Make it work. Then do it better - Andrei Straut

modified 13-Aug-12 12:22pm.

Thou mewling ill-breeding pignut!

AspDotNetDev13-Aug-12 6:25

AspDotNetDev

13-Aug-12 6:25

enhzflep wrote:
I just need to use your bank account to wire $50,000,000 out of this country, you will receive a generous 5%

There was a report not too long ago that studied why Nigerian scammers always say they are from Nigeria rather than some place not so famous for scamming. The reason was that if somebody is gullible enough to believe a scam that says it is from Nigaria, they are the most likely people to actually believe the scam. Essentially, being so obvious about their scam is their method to avoid talking to smart people, which would waste their time and cost them money. Big Grin | :-D

And I just found it: http://research.microsoft.com/pubs/167719/WhyFromNigeria.pdf

enhzflep13-Aug-12 6:34

13-Aug-12 6:34

And that my friend, is the most interesting piece of literature I've read all month.
Have a +5 for the article link.

Makes me wonder how much too little credit I've apportioned the writers of such emails.
It makes perfect sense.

Cheers, and thanks. Thumbs Up | :thumbsup:

Make it work. Then do it better - Andrei Straut

Bernhard Hiller14-Aug-12 21:24

Bernhard Hiller

14-Aug-12 21:24

Even worse. It is open to SQL injection attacks. A few days ago, someone asked in Q&A how a "UNION SELECT ALL null..." attack works, and someone pointed to Evil SQL[^]. Just tried a https://www.alcatraztrips.com/Confirmation.asp?order=179000%20having%201=1--[^]
which results in

Microsoft OLE DB Provider for SQL Server error '80040e14'
Column 'orders.OrderNumber' is invalid in the select list because it is not contained in either an aggregate function or the GROUP BY clause.
/Confirmation.asp, line 13

and some more bad injections... So easy to get the name of their db user etc.
But: please do not destroy their web site, just have fun!
It is such a great place to demonstrate the vulnerabilities of badly written code to our students here at CP.

Brisingr Aerowing15-Aug-12 15:03

Brisingr Aerowing

15-Aug-12 15:03

Bill Gates is a very rich man today... and do you want to know why? The answer is one word: versions.
Dave Barry
Read more at BrainyQuote[^]

ConvertToReadableNumber

Arpikusz8-Aug-12 21:57

Arpikusz

8-Aug-12 21:57

Can anyone tell me what this code does? Big Grin | :-D

I found it in a project I inherited:

public static string ConvertToReadableNumber(decimal num)
       {
           bool isNegative = num<0;
           num = Math.Floor(num);
           string numTxt = Math.Abs(num).ToString();
           string numReadable = "";
           int counter =0;
           for (int i = numTxt.Length-1; i >=0; i--)
           {
               if (counter > 0 && counter % 3 == 0)
                   numReadable = "," + numReadable;
               numReadable = numTxt[i] + numReadable;
               counter++;
           }
           if(isNegative)
               numReadable = "-"+numReadable;
           return numReadable;
       }

I'm guessing that a ToString() would have done the job. Smile | :)

Shameel8-Aug-12 22:33

Shameel

8-Aug-12 22:33

A blind programmer who reinvented the wheel (in a pretty much bad manner) without ever looking at the rich API the underlying framework has to offer. The best thing about using framework provided APIs is that we can safely assume (to some extent) that the code is tried and tested for all scenarios.

Reminds me of this[^], had the programmer used Date.AddYears() method, it would have saved a lot of headaches for Microsoft's customers, lot of customers to Microsoft and most importantly, would not have cost him/her job. At least, the programmer could have done a sanity check on the resultant date to ensure that it is valid.

J4amieC9-Aug-12 0:43

J4amieC

9-Aug-12 0:43

Its an abomination, for which the author should be sumarily executed.

It takes a decimal, floors it, and outputs it with commas as the thousand saparator:

http://rextester.com/CAYFY90912[^]

It boils down to this one-liner:

var result = Math.Floor(num).ToString("#,#");

Brisingr Aerowing9-Aug-12 14:46

Brisingr Aerowing

9-Aug-12 14:46

Cool site! And I agree!

Bill Gates is a very rich man today... and do you want to know why? The answer is one word: versions.
Dave Barry
Read more at BrainyQuote[^]

bojanh13-Aug-12 4:14

bojanh

13-Aug-12 4:14

If you really want to be precise and duplicate his function, it's actually this:

var result = Math.Floor(num).ToString("#,#", System.Globalization.CultureInfo.InvariantCulture);

Perhaps the author hard coded the thousand separator for some reason...

Bernhard Hiller9-Aug-12 0:55

Bernhard Hiller

9-Aug-12 0:55

The real wtf is the enormous variation in the paramters of the conversion function. When you know how to do it correctly with VB6, it won't work with C++, C#, Oracle SQL, MySQL, Microsoft SQL Server, ...
I guess the guy who wrote the code was a victim of that diversity, and wrote it after switching to C# from something else.

Eddy Vluggen9-Aug-12 1:06

Eddy Vluggen

9-Aug-12 1:06

ToString would have done a better job as it's not depending on the culture.

Lot's of companies that have utility-functions like this in their "toobox", often rebuilding what's already there.

To me, that proves that most of these companies are too rich. If they had to be careful on what they'd spend, sh*t like this would be gone.

Bastard Programmer from Hell Suspicious | :suss:

if you can't read my code, try converting it here[^]

Pete O'Hanlon9-Aug-12 1:47

Pete O'Hanlon

9-Aug-12 1:47

Arpikusz wrote:
numReadable = "," + numReadable;

What happens if the culture is one that uses . instead of , for the separator? Not only is this bad code, it's lazy bad code because it's not taking localisation into account.

Also, as I look over the code, what happens if num is 3000.27? I can't see anywhere in this code that actually adds the portion after the decimal on to numReadable.

*pre-emptive celebratory nipple tassle jiggle* - Sean Ewington

"Mind bleach! Send me mind bleach!" - Nagy Vilmos

CodeStash - Online Snippet Management | My blog | MoXAML PowerToys | Mole 2010 - debugging made easier