search

IDArlingTeam / IDArling

Collaborative Reverse Engineering plugin for IDA Pro & Hex-Rays
https://idarling.re
GNU General Public License v3.0
660 stars 99 forks source link

Can't select an existing project when saving #72

Closed roblabla closed 5 years ago

roblabla commented 6 years ago

I'm running an IDArling server, and I noticed that it was impossible to save a database to an existing project. Clicking on them does not seem to trigger the _project_clicked function. I couldn't figure out the reason behind it.

unknown

patateqbool commented 5 years ago

Hi, it's the same idb that you're trying to save on a remote project?

roblabla commented 5 years ago

It's an IDB i never uploaded. Can a project not have multiple distinct databases? The fact that each project has a table of databases made me think it was just an organization thing.

patateqbool commented 5 years ago

A project can have multiple versions, just upload one time, then click on the project to upload a new version (database) of this project.

roblabla commented 5 years ago

I cannot click on any project in the left "Project List" table. Only thing that works is clicking on "Create Project". If I click on any of the existing project (In the screenshot above, Switch300, FW100, w.e.), nothing happens.

I expected that clicking on a project would cause the database list to be populated, and the Create Database button to become available.

patateqbool commented 5 years ago

Excuse me, I was vague previously.

Projects are grayed out when the md5 of the remote file is not equal to the md5 of the database you want to upload: https://github.com/IDArlingTeam/IDArling/blob/a9f07a250c7f3e8427ba8275aea3d585c8a8f034/idarling/interface/dialogs.py#L294 This is an intended feature, as the server is shared between several people it avoids mistakenly erasing someone else's idb.

roblabla commented 5 years ago

OK, I think I understand. So a "Project" is tied to a single binary file, but can have multiple IDBs in it, all of them targeting the same file. Is that correct?

patateqbool commented 5 years ago

Yes.

patateqbool commented 5 years ago

Closing this issue, feel free to reopen it @roblabla if you still have questions.