The location was line 799 of ubercart/shipping/uc_ups/uc_ups.module. A loop was running from 0 to count() rather than count() - 1, causing an array bounds error. Patch to follow momentarily.

CommentFileSizeAuthor
#1 ubercart-undefined-offset-1430840-0.patch558 bytesjerry

Comments

jerry’s picture

jerry commented
Status: Active » Needs review
StatusFileSize
new ubercart-undefined-offset-1430840-0.patch558 bytes

Patch attached.

tr’s picture

tr commented
Priority: Normal » Minor
Status: Needs review » Fixed

Gee, that bug's been in Ubercart since Drupal 5! And it traces back to code taken from http://drupal.org/node/70915, which is almost 6 years old!

It hasn't been noticed before because E_NOTICE isn't set by default in Drupal 5 and 6, and in Drupal 7 the notice will only show if there were errors in the quote response returned from UPS. It's a purely cosmetic issue that doesn't affect the proper functioning of the module.

Committed the patch to both 7.x-3.x and 6.x-2.x.

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.