Skip to content

Commit

Permalink
[PATCH] fbcon: Sanitize fbcon
Browse files Browse the repository at this point in the history 
Do not pass the structure display since fbcon is already keeping the pointer
to the current display.

Signed-off-by: Antonino Daplas <adaplas@pol.net>
Signed-off-by: Andrew Morton <akpm@osdl.org>
Signed-off-by: Linus Torvalds <torvalds@osdl.org>
  • Loading branch information
Antonino A. Daplas authored and Linus Torvalds committed Jan 10, 2006
1 parent 8416131 commit b73deed
Show file tree
Hide file tree
Showing 8 changed files with 71 additions and 82 deletions.
14 changes: 7 additions & 7 deletions drivers/video/console/bitblit.c
View file
Original file line number Diff line number Diff line change
Expand Up @@ -234,14 +234,14 @@ static void bit_clear_margins(struct vc_data *vc, struct fb_info *info,
}
}

static void bit_cursor(struct vc_data *vc, struct fb_info *info,
struct display *p, int mode, int softback_lines, int fg, int bg)
static void bit_cursor(struct vc_data *vc, struct fb_info *info, int mode,
int softback_lines, int fg, int bg)
{
struct fb_cursor cursor;
struct fbcon_ops *ops = (struct fbcon_ops *) info->fbcon_par;
struct fbcon_ops *ops = info->fbcon_par;
unsigned short charmask = vc->vc_hi_font_mask ? 0x1ff : 0xff;
int w = (vc->vc_font.width + 7) >> 3, c;
int y = real_y(p, vc->vc_y);
int y = real_y(ops->p, vc->vc_y);
int attribute, use_sw = (vc->vc_cursor_type & 0x10);
int err = 1;
char *src;
Expand Down Expand Up @@ -310,7 +310,7 @@ static void bit_cursor(struct vc_data *vc, struct fb_info *info,
}

if (cursor.set & FB_CUR_SETSIZE ||
vc->vc_cursor_type != p->cursor_shape ||
vc->vc_cursor_type != ops->p->cursor_shape ||
ops->cursor_state.mask == NULL ||
ops->cursor_reset) {
char *mask = kmalloc(w*vc->vc_font.height, GFP_ATOMIC);
Expand All @@ -323,10 +323,10 @@ static void bit_cursor(struct vc_data *vc, struct fb_info *info,
kfree(ops->cursor_state.mask);
ops->cursor_state.mask = mask;

p->cursor_shape = vc->vc_cursor_type;
ops->p->cursor_shape = vc->vc_cursor_type;
cursor.set |= FB_CUR_SETSHAPE;

switch (p->cursor_shape & CUR_HWMASK) {
switch (ops->p->cursor_shape & CUR_HWMASK) {
case CUR_NONE:
cur_height = 0;
break;
Expand Down
49 changes: 23 additions & 26 deletions drivers/video/console/fbcon.c
View file
Original file line number Diff line number Diff line change
Expand Up @@ -209,13 +209,13 @@ static irqreturn_t fb_vbl_detect(int irq, void *dummy, struct pt_regs *fp)
#endif

#ifdef CONFIG_FRAMEBUFFER_CONSOLE_ROTATION
static inline void fbcon_set_rotation(struct fb_info *info, struct display *p)
static inline void fbcon_set_rotation(struct fb_info *info)
{
struct fbcon_ops *ops = info->fbcon_par;

if (!(info->flags & FBINFO_MISC_TILEBLITTING) &&
p->con_rotate < 4)
ops->rotate = p->con_rotate;
ops->p->con_rotate < 4)
ops->rotate = ops->p->con_rotate;
else
ops->rotate = 0;
}
Expand Down Expand Up @@ -265,7 +265,7 @@ static void fbcon_rotate_all(struct fb_info *info, u32 rotate)
fbcon_set_all_vcs(info);
}
#else
static inline void fbcon_set_rotation(struct fb_info *info, struct display *p)
static inline void fbcon_set_rotation(struct fb_info *info)
{
struct fbcon_ops *ops = info->fbcon_par;

Expand Down Expand Up @@ -402,7 +402,7 @@ static void fb_flashcursor(void *private)
c = scr_readw((u16 *) vc->vc_pos);
mode = (!ops->cursor_flash || ops->cursor_state.enable) ?
CM_ERASE : CM_DRAW;
ops->cursor(vc, info, p, mode, softback_lines, get_color(vc, info, c, 1),
ops->cursor(vc, info, mode, softback_lines, get_color(vc, info, c, 1),
get_color(vc, info, c, 0));
release_console_sem();
}
Expand Down Expand Up @@ -647,29 +647,27 @@ static void fbcon_prepare_logo(struct vc_data *vc, struct fb_info *info,
}

#ifdef CONFIG_FB_TILEBLITTING
static void set_blitting_type(struct vc_data *vc, struct fb_info *info,
struct display *p)
static void set_blitting_type(struct vc_data *vc, struct fb_info *info)
{
struct fbcon_ops *ops = info->fbcon_par;

ops->p = (p) ? p : &fb_display[vc->vc_num];
ops->p = &fb_display[vc->vc_num];

if ((info->flags & FBINFO_MISC_TILEBLITTING))
fbcon_set_tileops(vc, info, p, ops);
fbcon_set_tileops(vc, info);
else {
fbcon_set_rotation(info, ops->p);
fbcon_set_rotation(info);
fbcon_set_bitops(ops);
}
}
#else
static void set_blitting_type(struct vc_data *vc, struct fb_info *info,
struct display *p)
static void set_blitting_type(struct vc_data *vc, struct fb_info *info)
{
struct fbcon_ops *ops = info->fbcon_par;

info->flags &= ~FBINFO_MISC_TILEBLITTING;
ops->p = (p) ? p : &fb_display[vc->vc_num];
fbcon_set_rotation(info, ops->p);
ops->p = &fb_display[vc->vc_num];
fbcon_set_rotation(info);
fbcon_set_bitops(ops);
}
#endif /* CONFIG_MISC_TILEBLITTING */
Expand Down Expand Up @@ -697,7 +695,7 @@ static int con2fb_acquire_newinfo(struct vc_data *vc, struct fb_info *info,
if (!err) {
memset(ops, 0, sizeof(struct fbcon_ops));
info->fbcon_par = ops;
set_blitting_type(vc, info, NULL);
set_blitting_type(vc, info);
}

if (err) {
Expand Down Expand Up @@ -933,7 +931,7 @@ static const char *fbcon_startup(void)
ops->cur_rotate = -1;
info->fbcon_par = ops;
p->con_rotate = rotate;
set_blitting_type(vc, info, NULL);
set_blitting_type(vc, info);

if (info->fix.type != FB_TYPE_TEXT) {
if (fbcon_softback_size) {
Expand Down Expand Up @@ -1093,7 +1091,7 @@ static void fbcon_init(struct vc_data *vc, int init)

ops = info->fbcon_par;
p->con_rotate = rotate;
set_blitting_type(vc, info, NULL);
set_blitting_type(vc, info);

cols = vc->vc_cols;
rows = vc->vc_rows;
Expand Down Expand Up @@ -1141,9 +1139,9 @@ static void fbcon_init(struct vc_data *vc, int init)
if (vc == svc && softback_buf)
fbcon_update_softback(vc);

if (ops->rotate_font && ops->rotate_font(info, vc, p)) {
if (ops->rotate_font && ops->rotate_font(info, vc)) {
ops->rotate = FB_ROTATE_UR;
set_blitting_type(vc, info, p);
set_blitting_type(vc, info);
}

}
Expand Down Expand Up @@ -1243,7 +1241,6 @@ static void fbcon_cursor(struct vc_data *vc, int mode)
{
struct fb_info *info = registered_fb[con2fb_map[vc->vc_num]];
struct fbcon_ops *ops = info->fbcon_par;
struct display *p = &fb_display[vc->vc_num];
int y;
int c = scr_readw((u16 *) vc->vc_pos);

Expand All @@ -1260,7 +1257,7 @@ static void fbcon_cursor(struct vc_data *vc, int mode)
y = 0;
}

ops->cursor(vc, info, p, mode, y, get_color(vc, info, c, 1),
ops->cursor(vc, info, mode, y, get_color(vc, info, c, 1),
get_color(vc, info, c, 0));
vbl_cursor_cnt = CURSOR_DRAW_DELAY;
}
Expand Down Expand Up @@ -2111,12 +2108,12 @@ static int fbcon_switch(struct vc_data *vc)
fbcon_add_cursor_timer(info);
}

set_blitting_type(vc, info, p);
set_blitting_type(vc, info);
ops->cursor_reset = 1;

if (ops->rotate_font && ops->rotate_font(info, vc, p)) {
if (ops->rotate_font && ops->rotate_font(info, vc)) {
ops->rotate = FB_ROTATE_UR;
set_blitting_type(vc, info, p);
set_blitting_type(vc, info);
}

vc->vc_can_do_color = (fb_get_color_depth(&info->var, &info->fix)!=1);
Expand Down Expand Up @@ -2739,7 +2736,7 @@ static void fbcon_modechanged(struct fb_info *info)
return;

p = &fb_display[vc->vc_num];
set_blitting_type(vc, info, p);
set_blitting_type(vc, info);

if (CON_IS_VISIBLE(vc)) {
var_to_display(p, &info->var, info);
Expand Down Expand Up @@ -2781,7 +2778,7 @@ static void fbcon_set_all_vcs(struct fb_info *info)
continue;

p = &fb_display[vc->vc_num];
set_blitting_type(vc, info, p);
set_blitting_type(vc, info);
var_to_display(p, &info->var, info);
cols = FBCON_SWAP(ops->rotate, info->var.xres, info->var.yres);
rows = FBCON_SWAP(ops->rotate, info->var.yres, info->var.xres);
Expand Down
11 changes: 4 additions & 7 deletions drivers/video/console/fbcon.h
View file
Original file line number Diff line number Diff line change
Expand Up @@ -62,12 +62,10 @@ struct fbcon_ops {
int fg, int bg);
void (*clear_margins)(struct vc_data *vc, struct fb_info *info,
int bottom_only);
void (*cursor)(struct vc_data *vc, struct fb_info *info,
struct display *p, int mode, int softback_lines,
int fg, int bg);
void (*cursor)(struct vc_data *vc, struct fb_info *info, int mode,
int softback_lines, int fg, int bg);
int (*update_start)(struct fb_info *info);
int (*rotate_font)(struct fb_info *info, struct vc_data *vc,
struct display *p);
int (*rotate_font)(struct fb_info *info, struct vc_data *vc);
struct fb_var_screeninfo var; /* copy of the current fb_var_screeninfo */
struct timer_list cursor_timer; /* Cursor timer */
struct fb_cursor cursor_state;
Expand Down Expand Up @@ -173,8 +171,7 @@ struct fbcon_ops {
#define SCROLL_PAN_REDRAW 0x005

#ifdef CONFIG_FB_TILEBLITTING
extern void fbcon_set_tileops(struct vc_data *vc, struct fb_info *info,
struct display *p, struct fbcon_ops *ops);
extern void fbcon_set_tileops(struct vc_data *vc, struct fb_info *info);
#endif
extern void fbcon_set_bitops(struct fbcon_ops *ops);
extern int soft_cursor(struct fb_info *info, struct fb_cursor *cursor);
Expand Down
17 changes: 8 additions & 9 deletions drivers/video/console/fbcon_ccw.c
View file
Original file line number Diff line number Diff line change
Expand Up @@ -219,19 +219,18 @@ static void ccw_clear_margins(struct vc_data *vc, struct fb_info *info,
}
}

static void ccw_cursor(struct vc_data *vc, struct fb_info *info,
struct display *p, int mode, int softback_lines,
int fg, int bg)
static void ccw_cursor(struct vc_data *vc, struct fb_info *info, int mode,
int softback_lines, int fg, int bg)
{
struct fb_cursor cursor;
struct fbcon_ops *ops = (struct fbcon_ops *) info->fbcon_par;
struct fbcon_ops *ops = info->fbcon_par;
unsigned short charmask = vc->vc_hi_font_mask ? 0x1ff : 0xff;
int w = (vc->vc_font.height + 7) >> 3, c;
int y = real_y(p, vc->vc_y);
int y = real_y(ops->p, vc->vc_y);
int attribute, use_sw = (vc->vc_cursor_type & 0x10);
int err = 1, dx, dy;
char *src;
u32 vyres = GETVYRES(p->scrollmode, info);
u32 vyres = GETVYRES(ops->p->scrollmode, info);

if (!ops->fontbuffer)
return;
Expand Down Expand Up @@ -303,7 +302,7 @@ static void ccw_cursor(struct vc_data *vc, struct fb_info *info,
}

if (cursor.set & FB_CUR_SETSIZE ||
vc->vc_cursor_type != p->cursor_shape ||
vc->vc_cursor_type != ops->p->cursor_shape ||
ops->cursor_state.mask == NULL ||
ops->cursor_reset) {
char *tmp, *mask = kmalloc(w*vc->vc_font.width, GFP_ATOMIC);
Expand All @@ -323,10 +322,10 @@ static void ccw_cursor(struct vc_data *vc, struct fb_info *info,
kfree(ops->cursor_state.mask);
ops->cursor_state.mask = mask;

p->cursor_shape = vc->vc_cursor_type;
ops->p->cursor_shape = vc->vc_cursor_type;
cursor.set |= FB_CUR_SETSHAPE;

switch (p->cursor_shape & CUR_HWMASK) {
switch (ops->p->cursor_shape & CUR_HWMASK) {
case CUR_NONE:
cur_height = 0;
break;
Expand Down
17 changes: 8 additions & 9 deletions drivers/video/console/fbcon_cw.c
View file
Original file line number Diff line number Diff line change
Expand Up @@ -203,19 +203,18 @@ static void cw_clear_margins(struct vc_data *vc, struct fb_info *info,
}
}

static void cw_cursor(struct vc_data *vc, struct fb_info *info,
struct display *p, int mode, int softback_lines,
int fg, int bg)
static void cw_cursor(struct vc_data *vc, struct fb_info *info, int mode,
int softback_lines, int fg, int bg)
{
struct fb_cursor cursor;
struct fbcon_ops *ops = (struct fbcon_ops *) info->fbcon_par;
struct fbcon_ops *ops = info->fbcon_par;
unsigned short charmask = vc->vc_hi_font_mask ? 0x1ff : 0xff;
int w = (vc->vc_font.height + 7) >> 3, c;
int y = real_y(p, vc->vc_y);
int y = real_y(ops->p, vc->vc_y);
int attribute, use_sw = (vc->vc_cursor_type & 0x10);
int err = 1, dx, dy;
char *src;
u32 vxres = GETVXRES(p->scrollmode, info);
u32 vxres = GETVXRES(ops->p->scrollmode, info);

if (!ops->fontbuffer)
return;
Expand Down Expand Up @@ -287,7 +286,7 @@ static void cw_cursor(struct vc_data *vc, struct fb_info *info,
}

if (cursor.set & FB_CUR_SETSIZE ||
vc->vc_cursor_type != p->cursor_shape ||
vc->vc_cursor_type != ops->p->cursor_shape ||
ops->cursor_state.mask == NULL ||
ops->cursor_reset) {
char *tmp, *mask = kmalloc(w*vc->vc_font.width, GFP_ATOMIC);
Expand All @@ -307,10 +306,10 @@ static void cw_cursor(struct vc_data *vc, struct fb_info *info,
kfree(ops->cursor_state.mask);
ops->cursor_state.mask = mask;

p->cursor_shape = vc->vc_cursor_type;
ops->p->cursor_shape = vc->vc_cursor_type;
cursor.set |= FB_CUR_SETSHAPE;

switch (p->cursor_shape & CUR_HWMASK) {
switch (ops->p->cursor_shape & CUR_HWMASK) {
case CUR_NONE:
cur_height = 0;
break;
Expand Down
9 changes: 4 additions & 5 deletions drivers/video/console/fbcon_rotate.c
View file
Original file line number Diff line number Diff line change
Expand Up @@ -18,8 +18,7 @@
#include "fbcon.h"
#include "fbcon_rotate.h"

static int fbcon_rotate_font(struct fb_info *info, struct vc_data *vc,
struct display *p)
static int fbcon_rotate_font(struct fb_info *info, struct vc_data *vc)
{
struct fbcon_ops *ops = info->fbcon_par;
int len, err = 0;
Expand All @@ -28,12 +27,12 @@ static int fbcon_rotate_font(struct fb_info *info, struct vc_data *vc,
u8 *dst;

if (vc->vc_font.data == ops->fontdata &&
p->con_rotate == ops->cur_rotate)
ops->p->con_rotate == ops->cur_rotate)
goto finished;

src = ops->fontdata = vc->vc_font.data;
ops->cur_rotate = p->con_rotate;
len = (!p->userfont) ? 256 : FNTCHARCNT(src);
ops->cur_rotate = ops->p->con_rotate;
len = (!ops->p->userfont) ? 256 : FNTCHARCNT(src);
s_cellsize = ((vc->vc_font.width + 7)/8) *
vc->vc_font.height;
d_cellsize = s_cellsize;
Expand Down
19 changes: 9 additions & 10 deletions drivers/video/console/fbcon_ud.c
View file
Original file line number Diff line number Diff line change
Expand Up @@ -249,20 +249,19 @@ static void ud_clear_margins(struct vc_data *vc, struct fb_info *info,
}
}

static void ud_cursor(struct vc_data *vc, struct fb_info *info,
struct display *p, int mode, int softback_lines,
int fg, int bg)
static void ud_cursor(struct vc_data *vc, struct fb_info *info, int mode,
int softback_lines, int fg, int bg)
{
struct fb_cursor cursor;
struct fbcon_ops *ops = (struct fbcon_ops *) info->fbcon_par;
struct fbcon_ops *ops = info->fbcon_par;
unsigned short charmask = vc->vc_hi_font_mask ? 0x1ff : 0xff;
int w = (vc->vc_font.width + 7) >> 3, c;
int y = real_y(p, vc->vc_y);
int y = real_y(ops->p, vc->vc_y);
int attribute, use_sw = (vc->vc_cursor_type & 0x10);
int err = 1, dx, dy;
char *src;
u32 vyres = GETVYRES(p->scrollmode, info);
u32 vxres = GETVXRES(p->scrollmode, info);
u32 vyres = GETVYRES(ops->p->scrollmode, info);
u32 vxres = GETVXRES(ops->p->scrollmode, info);

if (!ops->fontbuffer)
return;
Expand Down Expand Up @@ -334,7 +333,7 @@ static void ud_cursor(struct vc_data *vc, struct fb_info *info,
}

if (cursor.set & FB_CUR_SETSIZE ||
vc->vc_cursor_type != p->cursor_shape ||
vc->vc_cursor_type != ops->p->cursor_shape ||
ops->cursor_state.mask == NULL ||
ops->cursor_reset) {
char *mask = kmalloc(w*vc->vc_font.height, GFP_ATOMIC);
Expand All @@ -347,10 +346,10 @@ static void ud_cursor(struct vc_data *vc, struct fb_info *info,
kfree(ops->cursor_state.mask);
ops->cursor_state.mask = mask;

p->cursor_shape = vc->vc_cursor_type;
ops->p->cursor_shape = vc->vc_cursor_type;
cursor.set |= FB_CUR_SETSHAPE;

switch (p->cursor_shape & CUR_HWMASK) {
switch (ops->p->cursor_shape & CUR_HWMASK) {
case CUR_NONE:
cur_height = 0;
break;
Expand Down
Loading

0 comments on commit b73deed

Please sign in to comment.