bugAVR Downloader/UploaDEr - Bugs: bug #21076, -vvvv serial receive prints are...

 
 

You are not allowed to post comments on this tracker with your current authentification level.

bug #21076: -vvvv serial receive prints are empty in Win32 build

Submitted by:  Joerg Wunsch <joerg_wunsch>
Submitted on:  Fri Sep 14 09:04:36 2007  
 
Category: NoneSeverity: 5 - Blocker
Priority: 9 - ImmediateItem Group: None
Status: FixedPrivacy: Public
Assigned to: Eric Weddington <arcanum>Originator Name: Joerg Wunsch
Open/Closed: ClosedRelease: None
Programmer hardware: Device type: 

Mon Oct 29 23:12:06 2007, comment #3:

Confirmed this fixes it - old version of avrdude had the bug, newly checked out CVS did not. Cha-Ching.

Colin O Flynn <c_oflynn>
Project Member
Mon Oct 29 21:52:31 2007, comment #2:

I've chosen a different way to fix it than in the suggested
patch.

Eric, please close the bug after verifying it really does the
job -- but I'm pretty positive it does, after reading the
MSDN documentation for ReadFile().

Joerg Wunsch <joerg_wunsch>
Project Administrator
Fri Sep 14 14:47:02 2007, comment #1:

Eric, curious, since you uploaded a patch, I assume this means
you've already tested it? That would be great, I could only
have fixed it blindly myself.

Joerg Wunsch <joerg_wunsch>
Project Administrator
Fri Sep 14 09:04:36 2007, original submission:

It turns out the variable "len" which is used to control the
loop is never assigned.

Good item to demonstrate why you'd rather not blindly
initialize all local variables by default in their definition,
like:

static int ser_recv(union filedescriptor fd, unsigned char buf, size_t buflen)
{
unsigned char c;
unsigned char * p = buf;
size_t len = 0;
^^^^

Joerg Wunsch <joerg_wunsch>
Project Administrator

 

Attached Files
file #13943:  avrdude-bug-21076.patch added by arcanum (437B - application/octet-stream - Untested patch.)

 

Depends on the following items: None found

Items that depend on this one: None found

 

Carbon-Copy List
  • -unavailable- added by c_oflynn (Posted a comment)
  • -unavailable- added by arcanum (Updated the item)
  • -unavailable- added by joerg_wunsch (Submitted the item)
  •  

    Do you think this task is very important?
    If so, you can click here to add your encouragement to it.
    This task has 0 encouragements so far.

    Only logged-in users can vote.

     

    Please enter the title of George Orwell's famous dystopian book (it's a date):

     

     

    Follow 6 latest changes.

    Date Changed By Updated Field Previous Value => Replaced By
    Mon Oct 29 23:12:06 2007c_oflynnOpen/ClosedOpen=>Closed
    Mon Oct 29 21:52:31 2007joerg_wunschStatusNone=>Fixed
    Fri Sep 14 17:15:57 2007arcanumAssigned toNone=>arcanum
    Fri Sep 14 14:45:15 2007arcanumSeverity3 - Normal=>5 - Blocker
      Priority5 - Normal=>9 - Immediate
    Fri Sep 14 14:44:41 2007arcanumAttached File-=>Added avrdude-bug-21076.patch, #13943

    Back to the top


    Powered by Savane 3.1-cleanup