After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 143618 - gdmphotosetup should offer to resize images
gdmphotosetup should offer to resize images
Status: RESOLVED FIXED
Product: gdm
Classification: Core
Component: general
2.4.4.x
Other Linux
: Normal enhancement
: ---
Assigned To: GDM maintainers
GDM maintainers
Depends on:
Blocks:
 
 
Reported: 2004-06-02 22:58 UTC by John Spray
Modified: 2005-03-10 20:31 UTC
See Also:
GNOME target: ---
GNOME version: 2.9/2.10



Description John Spray 2004-06-02 22:58:19 UTC
Rather than simply complaining to the user that their file is too big,
gdmphotosetup should offer to resize the graphic before writing it to
~/.gnome/photo.

Failing that, the dialog at least should not assume that the user knows how to
resize an image, alter the compression level, or even has a concept of the
relationship between an images dimensions/format and the logical size of the image.
Comment 1 Brian Cameron 2005-02-03 22:44:11 UTC

*** This bug has been marked as a duplicate of 151784 ***
Comment 2 John Spray 2005-02-03 23:08:53 UTC
Bug 151784 has nothing to do with this one: it refers to the display of the
default face icon at this login screen.  This one refers to choosing a face
graphic in gdmfacechooser, where the user may receive a message like "Picture is
too large\n\nThe system administrator disallowed pictures larger than 65536
bytes to show in the face browser".
Comment 3 Brian Cameron 2005-02-08 21:39:23 UTC
Thanks, you are right.

The gdmphotosetup should resize images.  the daemon should not, the daemon 
should do the least amount of work and should really ignore large photos. 
That's a security issue, you don't want an evil or excessively long image 
dosing the daemon or the greeter.  This bug has to do with the gdmphotosetup
proggie that sets the picture.
Comment 4 Brian Cameron 2005-03-10 20:31:24 UTC
Updated in CVS head to scale images down to max icon height and width
automatically.