Work at SourceForge, help us to make it a better place! We have an immediate need for a Support Technician in our San Francisco or Denver office.

Close

PasswordSafe with 64 bit Linux

2010-11-21
2013-07-17
1 2 > >> (Page 1 of 2)
  • I've been happily using the Linux beta with 32 bit Linux, but thought Wine
    might be a better option for a 64 OS. So far, so good: I have 3.23 running
    well and life is once again good (ish), but one minor problem is the absence
    of Help. It opens fine and displays the left-hand pane, but the right-hand
    pane is blank and also the icons all appear to be a standard default.

    My guess is that this is a Wine issue, but I thought it worth asking here in
    case there is an obvious solution?

    Thanks to anyone.

    Peter HB

     
  • Wine's hh tool doesn't fully support chm format (you can see "fixme" messages
    when running wine hh ./pwsafe.chm in console. There are some discussions
    about it in winehq forum).

    Empty right pane possibly means, that you doesn't have
    Gecko installed (you can see message about it
    in console output of wine hh ./pwsafe.chm)

    Gecko can be installed manually, or with
    winetricks).

    After installing Gecko, text on the right pane is displayed, but there are no
    pictures.

    This can be solved by:

    • using external chm viewer (xchm, okular, ...)
    • installing ie6 using winetricks

    Sorry for my English

    PS: I've checked it under wine 1.3.7.

     
  • Many thanks for your help, which led to a solution. I upgraded wine to 1.3.8
    and used winetricks to install gecko and ie8. Lots of "fixme"s, but something
    in that process has done the trick and I now have text and pictures in
    Password Safe help.

    I'll try to be a little more scientific on another machine and refine what the
    fix actually was.

     
  • Evan Black
    Evan Black
    2011-07-02

    Same here I'm running PWS v.3.26 in wine ubuntu 11.04 x64. It works ok, but
    would love a native 64 bit compile.

     

  • Anonymous
    2011-07-20

    Just a question... why not go ahead and compile it for yourself?

    You might actually have fun learning a new skill :~)

     
  • scad
    scad
    2012-01-11

    to https://www.google.com/accounts :-)

    What about version 0.7 for ubuntu x64?

     

  • Anonymous
    2012-01-16

    Hi Scad18,

    I did write to developer about this. He said he would update very soon. I
    guess he is busy with other things.

     
  • Brian
    Brian
    2012-01-23

    I went ahead and built this from source per the instructions in the tarball. I
    upgraded it on my Lucid 64bit from 0.6.0 and tested it. I will share it with
    those who would like a copy unless the developer would rather I not.

    http://dl.dropbox.com/u/39638663/ubuntu/passwordsafe-
    ubuntu-0.7.0BETA.amd64.deb

     
  • Rony Shapiro
    Rony Shapiro
    2012-01-23

    Thanks for sharing your build.

    Did you make any changes to the source to get it to compile? If so, please
    send me the patch, so that I can add the changes to the repository.

    Rony

     
  • Brian
    Brian
    2012-01-23

    Hi Rony,

    No changes to the source were made. I just followed your instructions in the
    README.LINUX.DEVELOPERS.txt file and built it on a vanilla 10.04.3 64bit box.

    Thank you for making this available.

    Brian

     
  • Micha
    Micha
    2012-05-01

    Just tried compiling trunk (svn4885) under 64-bit Debian and had to make the
    two following changes.

    1. Won't compile due to invalid cast.

    $ svn diff src/ui/wxWidgets/addeditpropsheet.cpp

    Index: src/ui/wxWidgets/addeditpropsheet.cpp
    ===================================================================
    --- src/ui/wxWidgets/addeditpropsheet.cpp   (revision 4885)
    +++ src/ui/wxWidgets/addeditpropsheet.cpp   (working copy)
    @@ -1126,14 +1126,16 @@
     static short GetSelectedDCA(wxComboBox *pcbox, short defval)
     {
       int sel = pcbox->GetSelection();
    -  short retval;
    +  intptr_t retval = -1;
    +  
       if (sel == wxNOT_FOUND) { // no selection - is this possible with our combobox?
    -    return -1;
    +   goto done;
       } else {
    -    retval = reinterpret_cast<int>(pcbox->GetClientData(sel));
    +    retval = reinterpret_cast<intptr_t>(pcbox->GetClientData(sel));
       }
       if (retval == defval)
         retval = -1;
    +done:
       return retval;
     }
    
    1. segfault when creating a new password database due to double-free:

    $ svn diff src/core/PWSfile.cpp

    Index: src/core/PWSfile.cpp
    ===================================================================
    --- src/core/PWSfile.cpp    (revision 4885)
    +++ src/core/PWSfile.cpp    (working copy)
    @@ -143,8 +143,10 @@
    
     int PWSfile::Close()
     {
    -  delete m_fish;
    -  m_fish = NULL;
    +  if (m_fish != NULL) {
    +   delete m_fish;
    +   m_fish = NULL;
    +  }
       if (m_fd != NULL) {
         fflush(m_fd);
         fclose(m_fd);
    
    • The PwsfileV3::Close() method of PwsfileV3 calls Pwsfile::Close() multiple times.
     
  • Rony Shapiro
    Rony Shapiro
    2012-05-02

    Thanks for the patches. I've a problem with the second one, though, as C++
    guarantees that delete-ing a null vary is safe and a no-op, so I don't see how
    the if statement you've added protects against a double-free. Perhaps the
    memory corruption's occurring elsewhere?

     
  • Micha
    Micha
    2012-05-02

    Hi Ronys,

    Wow, I'm rustier than I thought with my C++.

    I reverted the second patch and can no longer repro the segfault which is a
    bit odd as I was getting it constantly on startup. Even tried deleting my
    .pwsafe directory. Well, nevermind then.

    Cheers,

    • Micha.
     
  • DrK
    DrK
    2012-05-02

    Hi,

    With your first patch, would this one-line change be good enough? (i.e.
    changing <int> to <short>)

    From:

    retval = reinterpret_cast<int>(pcbox->GetClientData(sel));
    

    to:

    retval = reinterpret_cast<short>(pcbox->GetClientData(sel));
    

    David

     
  • Micha
    Micha
    2012-05-02

    No, you can't cast a pointer to an integer shorter than a pointer, even though
    you can cast or assign integers to shorter integers (C++ is weird). This is
    why I used uintptr_t as that is always guaranteed to be the same size as a
    pointer. But you can assign the result of the cast directly to the short :

    retval = reinterpret_cast<uintptr_t>(pcBox->GetClientData(sel));
    

    Since you're controlling what goes into the combo-box, you're sure to only
    ever get shorts out, otherwise you might also want to check for overflows.

    Ie,

    uintptr_t ptrVal = reinterpret_cast<uintptr_t>(pcBox->GetClientData(sel));
    retval = ptrVal;
    if (retval != ptrVal) /* Value overflow error */
    
     
  • DrK
    DrK
    2012-05-02

    How about the old-fashioned way (and yes - we know the value is only a short -
    in fact way less than 16 (decimal) in the current version?

    retval = (short)pcBox->GetClientData(sel); 
    [\code]
    
    David
    
     
  • DrK
    DrK
    2012-05-02

    Sorry - formatting issue:

    retval = (short)pcBox->GetClientData(sel);
    

    David

     
  • Micha
    Micha
    2012-05-02

    What was wrong with reinterpret_casting to an intptr_t and then assigning to
    retval? That keeps it C++ and type-safe and is also the smallest change from
    the existing code.

    Anyway, I don't care as long as it compiles and runs :)

     
  • Rony Shapiro
    Rony Shapiro
    2012-05-02

    Simplest version of patch that compiles cleanly & runs under 64bit has been
    committed (4886).

    Thanks,

    Rony

     
  • Micha
    Micha
    2012-05-05

    Cool, thanks Rony :)

     
  • Martin Barlow
    Martin Barlow
    2012-06-02

    I successfully built passwordsafe-debian-0.7.0BETA.amd64.deb on debian 6.05.
    Seemed to require no modifications. works great. thanks.

     
  • Brian
    Brian
    2012-08-31

    Please forgive the noob question in advance, but is it expected to have the
    amd64 deb package request an additional 240 packages (all of which are i386)?
    I'm sure they are dependencies, but some packages it is requesting seem
    strange. Like "bluez-alsa:i386" and "libgstreamer-plugins:i386".

    Also, this happens whether I compile 0.7.0BETA or 0.8.0BETA and on two
    different machines.

    Thanks

     
  • Brian
    Brian
    2012-08-31

    For those who want to compile on LinuxMint, here are the two items I changed
    to get past the compile errors:

    In the Makefile inside ./pwsafe-0.8BETA/install/deb change line 7 from:

    DISTRO := $(shell echo $(word 2,$(shell lsb_release -d))|tr [:upper:] [:lower:])
    

    to

    DISTRO := $(shell echo $(word 3,$(shell lsb_release -i))|tr [:upper:] [:lower:])
    

    and

    cp control-ubuntu.amd64 control-linuxmint.amd64
    
     
  • Brian
    Brian
    2012-08-31

    Regarding my noob question - I may have answered this myself and am a bit
    embarrassed.

     
1 2 > >> (Page 1 of 2)