Skip to content

Commit

Permalink
[media] cx25821: Remove bad strcpy to read-only char*
Browse files Browse the repository at this point in the history
The strcpy was being used to set the name of the board.
This was both wrong and redundant,
since the destination char* was read-only and
the name is set statically at compile time.

The type of the name field is changed to const char*
to prevent future errors.

Reported-by: Radek Masin <radek@masin.eu>
Signed-off-by: Ezequiel Garcia <elezegarcia@gmail.com>
Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>
  • Loading branch information
Ezequiel García authored and Mauro Carvalho Chehab committed Jul 31, 2012
1 parent eabdc68 commit c854d88
Show file tree
Hide file tree
Showing 2 changed files with 1 addition and 4 deletions.
3 changes: 0 additions & 3 deletions drivers/media/video/cx25821/cx25821-core.c
Original file line number Diff line number Diff line change
Expand Up @@ -904,9 +904,6 @@ static int cx25821_dev_setup(struct cx25821_dev *dev)
list_add_tail(&dev->devlist, &cx25821_devlist);
mutex_unlock(&cx25821_devlist_mutex);

strcpy(cx25821_boards[UNKNOWN_BOARD].name, "unknown");
strcpy(cx25821_boards[CX25821_BOARD].name, "cx25821");

if (dev->pci->device != 0x8210) {
pr_info("%s(): Exiting. Incorrect Hardware device = 0x%02x\n",
__func__, dev->pci->device);
Expand Down
2 changes: 1 addition & 1 deletion drivers/media/video/cx25821/cx25821.h
Original file line number Diff line number Diff line change
Expand Up @@ -186,7 +186,7 @@ enum port {
};

struct cx25821_board {
char *name;
const char *name;
enum port porta;
enum port portb;
enum port portc;
Expand Down

0 comments on commit c854d88

Please sign in to comment.