The Linux port segfaults if run with a different GS plugin then selected at startup

[GoogleCodeExporter]

What steps will reproduce the problem?
1. Go into the configuration screen.
2. Change Graphical plugins.
3. Choose Run Cd.

On ZeroGS, it segfaults on glGenFramebuffersEXT( 1, &s_uFramebuffer); in
the function ZeroGS::Create. If run with a debug build, it asserts due to
GSIrq being null.

It is hard to tell when this bug was introduced, as until recently, there
was unlikely to be more then one graphical plugin in pcsx2 on the Linux
port. Pre r560.

Per Zeydlitz, it is segfaulting due to glGenFramebuffersEXT being 0,
although this extension is mapped.

Until this is resolved, you will need to quit pcsx2 after changing gs
plugins before running the emulator.

Original issue reported on code.google.com by arcum42@gmail.com on 28 Feb 2009 at 9:50

[GoogleCodeExporter]

Ok. There is something need to be clean:

If we do following: change GS plugin, EXIT, than run, everyting is o'k. 
So incorrect code in closing section.

Original comment by Zeydl...@gmail.com on 2 Mar 2009 at 8:32

[GoogleCodeExporter]

Arcum, there is very itchy code on plugin handling. When we open configuration, 
we
open (dlopen) all of the plugins at FindPlugins, but never closed any of them. 
It
bad, and cause this issue. So this is 1-sting patch for you.

Original comment by Zeydl...@gmail.com on 2 Mar 2009 at 5:32

Attachments:

• good.diff

[GoogleCodeExporter]

Yeah, that is bad. That's one particular function that I haven't touched much,
either, so this issue must have been in there forever. I'd been looking at the 
actual
configuration function instead. :(

That means it must have had two graphical plugins loaded at the same time, which
explains a lot.

That whole files due for a rewrite, actually. I got halfway through one when 
tackling
the ini problem, then scrapped it, and decided to wait till later on it. That 
code is
half the reason why I started chopping up GtkGui and LnxMain into multiple 
files in
the first place.

Anyways, I'll go ahead and commit that now...

Original comment by arcum42@gmail.com on 2 Mar 2009 at 8:27

[GoogleCodeExporter]

Original comment by arcum42@gmail.com on 2 Mar 2009 at 8:29

• Changed state: Fixed