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 151784 - [PATCH] The greeter does not obey MaxIcon* settings with the default face
[PATCH] The greeter does not obey MaxIcon* settings with the default face
Status: VERIFIED WONTFIX
Product: gdm
Classification: Core
Component: general
2.6.0.x
Other Linux
: Normal normal
: ---
Assigned To: GDM maintainers
GDM maintainers
Depends on:
Blocks:
 
 
Reported: 2004-09-03 20:25 UTC by Diego González
Modified: 2009-08-15 18:40 UTC
See Also:
GNOME target: ---
GNOME version: ---

Attachments
Patch for bug described (1.86 KB, patch)
2004-09-03 20:26 UTC, Diego González 		none Details | Review

Description Diego González 2004-09-03 20:25:27 UTC 
as summary says
Comment 1 Diego González 2004-09-03 20:26:14 UTC 
Created attachment 31246 [details] [review]
Patch for bug described
Comment 2 Brian Cameron 2005-02-03 22:44:12 UTC 
*** Bug 143618 has been marked as a duplicate of this bug. ***
Comment 3 Brian Cameron 2005-02-08 21:41:21 UTC 
According to George, the gdmphotosetup should ensure that the images are
set to the right size, 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.

See bug 143618 (now reopened) which is where this bug should be fixed.
Want to create a patch for gdmphotosetup instead?
Comment 4 Brian Cameron 2005-02-15 20:41:17 UTC 
Apologies, after talking to George, he clarified that this patch scales
the default icon, not the user icon.  The user icons should not be scaled
in GDM2, but it is okay for the default icon (which only root should be able
to install) makes sense.
Comment 5 Brian Cameron 2005-02-15 20:45:19 UTC 
Fixed in CVS head.