Project

General

Profile

Actions

Emulator Issues #6288

closed

Access Violation when making a new memory card.

Added by Lioncash almost 11 years ago.

Status:
Fixed
Priority:
High
Assignee:
% Done:

0%

Operating system:
N/A
Issue type:
Bug
Milestone:
Regression:
No
Relates to usability:
Yes
Relates to performance:
No
Easy:
No
Relates to maintainability:
No
Regression start:
Fixed in:

Description

Steps to reproduce:

  1. Open Dolphin and go to Tools -> Memcard Manager (GC)

  2. Hit browse

  3. In the file name box, enter a file name that is not present in the current directory that is being viewed.

  4. Dolphin will kick up a prompt asking if you would like to create a new memcard. Hit "Yes"

  5. Another prompt comes up asking if you would like to format it as ASCII or SHIFT-JIS. Format it as ASCII.

  6. It will now crash.


I've tried to solve this problem today but came up with nothing.

Though, when running in Visual Studio, it breaks at line 338 in GCMemCard.cpp

"if (BE32(CurrentDir->Dir[i].Gamecode)!= 0xFFFFFFFF)"

I have a feeling either CurrentDir or Dir is not being initialized correctly. This is also evidenced by the fact that when I open up the info for CurrentDir in Visual Studio, there is "<bad ptr>" tagged on almost everything.

Actions #1

Updated by Lioncash almost 11 years ago

To provide extra info which may help.

The GetNumFiles() call which causes all of this is called on line 631 of MemcardManager.cpp.

Actions #2

Updated by lpfaint99 almost 11 years ago

I'll take a look, I'm sure it was working the last time I made a change to it

Actions #3

Updated by lpfaint99 almost 11 years ago

i wasn't able to identify when it started, but it's an easy fix.

Actions #4

Updated by Lioncash almost 11 years ago

Seems like it still doesn't fix it. I still get this when I try to make a memory card manually:

http://i.imgur.com/GYEwxVx.png

Actions #6

Updated by lpfaint99 almost 11 years ago

if the CurrentDir pointer is bad, everything inside it will be garbage.

it works just fine with builds compiled myself, and the buildbot on dolphin-emu.org
i tested both 32 and 64 bit
can you reproduce it with the builds from the website? any special setup you have/changes to the gcmemcard source?

Actions #7

Updated by Lioncash almost 11 years ago

Ah, I suppose that error only gets thrown when debugging in VS or something. It doesn't happen when I run it without the debugger.

Actions #8

Updated by Lioncash almost 11 years ago

Ah ok, got it to work. Must have forgot to do a rebuild or something. It's all good now; sorry about that.

Actions #9

Updated by lpfaint99 almost 11 years ago

Great, I couldn't get it to occur even with the debugger running

Actions #10

Updated by rachelbryk almost 11 years ago

  • Status changed from Invalid to Fixed

This definitely used to happen, but seems fixed now.

Actions

Also available in: Atom PDF