search

codebackup / bwapi

Automatically exported from code.google.com/p/bwapi
0 stars 0 forks source link

BWAPI doesn't inject in Win7 without elevated privileges #383

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago 
What steps will reproduce the problem?
1.  Install the current BWAPI Head revision on Windows 7
2.  Try and run it without administrator privileges
3.

What is the expected output? What do you see instead?
It *should* run normally, instead it says "Injection failed.  This usually 
happens because BWAPI crashed before injection completed"  (or something to 
that effect)

What version of the product are you using? On what operating system?
BWAPI 3.5.4 and onwards (up to current Head at r3664

On Windows 7, 64bit  without administrator privileges

Please provide any additional information below.
I tracked down an old revision of BWAPI being used for a replay project 
http://www.teamliquid.net/forum/viewmessage.php?topic_id=111664

and the BWAPI.dll included with that is injected just fine with the r3664 
chaoslauncher injector.

Original issue reported on code.google.com by earthlin...@gmail.com on 24 Apr 2011 at 9:23

GoogleCodeExporter commented 9 years ago 
What I forgot to mention is that the old revision that is known to work was 
r1914.

Original comment by earthlin...@gmail.com on 24 Apr 2011 at 9:27

GoogleCodeExporter commented 9 years ago

Original comment by AHeinerm on 26 Apr 2011 at 12:11

GoogleCodeExporter commented 9 years ago

Original comment by AHeinerm on 26 Apr 2011 at 12:14

GoogleCodeExporter commented 9 years ago 
I narrowed down the changes to r2176 as the last good revision.

The only files that changed between r2176 and r2177 are

bwapi/BWAPI/Source/BWAPI/Server.cpp
bwapi/ExampleAIClient/Source/ExampleAIClient.cpp
bwapi/SharedStructures/PlayerData.h

Original comment by earthlin...@gmail.com on 30 Apr 2011 at 2:22

GoogleCodeExporter commented 9 years ago 
I don't see anything that could cause a problem...?

Original comment by AHeinerm on 30 Apr 2011 at 2:48

GoogleCodeExporter commented 9 years ago 
Edit... binary search fail.  Or maybe just me.  It was actually r2170 to r2171. 
  r2170 is definitely good.  I'm applying the file changes individually now to 
narrow it further.

Original comment by earthlin...@gmail.com on 30 Apr 2011 at 5:15

GoogleCodeExporter commented 9 years ago

Original comment by AHeinerm on 30 Apr 2011 at 6:17

GoogleCodeExporter commented 9 years ago

Original comment by AHeinerm on 30 Apr 2011 at 6:18

GoogleCodeExporter commented 9 years ago 
Issue resolved for us (at WWU).  Fix was essentially removing the calls to 
Util::Logger::globalLog->log  in the server::server() constructor.

Original comment by earthlin...@gmail.com on 30 Apr 2011 at 6:23

GoogleCodeExporter commented 9 years ago 
You mentioned another problem though? What was it?
I removed all the Logger stuff in the head rev.

Original comment by AHeinerm on 30 Apr 2011 at 4:08

GoogleCodeExporter commented 9 years ago 
Our other problems are related to our bot, and the fact that the starcraft 
installpath in my registry was wrong.  :D  We're all up and running now.

Original comment by earthlin...@gmail.com on 30 Apr 2011 at 8:34