Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Nemo Patched ! #3

Merged
merged 9 commits into from
Aug 11, 2012
Merged

Nemo Patched ! #3

merged 9 commits into from
Aug 11, 2012

Conversation

ikeydoherty
Copy link
Contributor

Howdy do Gentlemen :)
As per: https://plus.google.com/105704906416776356848/posts/L1FnY1gc365
I want you guys to use my changes. Its the only way to keep them alive properly instead of rebasing on 3.6 etc. We're chasing the same ball so I figured we may as well be working from the same base.
I've made a point of publically telling people I pushed my patches against Nemo.
Cooperation! :) Serious don't be tied by my patches, extend, modify :)
Hope you use, happy to take suggestions and modify as per your requests ^^

@clefebvre
Copy link
Member

Hi Ikey,

Great to join forces and with very similar ideas. With Gnome going crazy I think we're indeed chasing the same ball here. We're talking to glebihan right now and seeing if we can revert his changes (entry bar basically) and merging yours. Give me a few days for it to happen (you're hitting us right in the middle of August here!!) :)

@ikeydoherty
Copy link
Contributor Author

I was kinda curious about that commit tbh :)
I personally think it would be better to make it so that you click on the pathbar to show the location entry like
in Dolphin. We'll see :) In the mean time I'll set about reverting glebihan's commit so you can just merge my
pull request with no damage.
And yeah, kinda cool to be joining forces again ^^

@glebihan
Copy link
Member

Don't lose time, I've already made the changes to revert my previous commits.
I'll commit shortly.

@glebihan glebihan merged commit b9721f8 into linuxmint:master Aug 11, 2012
@ikeydoherty
Copy link
Contributor Author

Thanks

@4C3 4C3 mentioned this pull request Jan 2, 2013
rik-shaw referenced this pull request Jul 26, 2014
@Zerophase Zerophase mentioned this pull request Jan 12, 2016
mtwebster pushed a commit that referenced this pull request Mar 11, 2021
With some GIO backends (observed at least with sftp),
g_file_info_get_symlink_target() returns an empty string rather
than NULL for non-symlinks. A warning and then segfault result:

    ** (nemo:8386): WARNING **: 18:34:22.807: File has symlink target, but  is not marked as symlink

    Thread 1 "nemo" received signal SIGSEGV, Segmentation fault.
    0x00007ffff71beac0 in g_str_hash () from /usr/lib/libglib-2.0.so.0
    (gdb) bt
    #0  0x00007ffff71beac0 in g_str_hash () at /usr/lib/libglib-2.0.so.0
    #1  0x00007ffff71bfa2f in g_hash_table_lookup_extended () at /usr/lib/libglib-2.0.so.0
    #2  0x000055555564926e in modify_link_hash_table at ../libnemo-private/nemo-file.c:600
    #3  0x000055555564ed11 in add_to_link_hash_table at ../libnemo-private/nemo-file.c:642
    #4  update_info_internal at ../libnemo-private/nemo-file.c:2683
    #5  0x000055555564f767 in nemo_file_update_info  at ../libnemo-private/nemo-file.c:2702
    #6  0x00005555556263ec in query_info_callback at ../libnemo-private/nemo-directory-async.c:3230

Defend against this at two levels:
 - For non-symlinks, don't call g_file_info_get_symlink_target().
 - Use g_return_if_fail() to prevent NULL pointer dereference.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants