diff options
author | Shaun Ruffell <sruffell@digium.com> | 2011-06-02 20:02:47 +0000 |
---|---|---|
committer | Shaun Ruffell <sruffell@digium.com> | 2011-06-02 20:02:47 +0000 |
commit | ce9d8dd99f5c88cd964ca34e1b17ba2ce17509a5 (patch) | |
tree | ab99d152522405d0e6019771f623fb0da324b7fd /drivers | |
parent | d3b38dd3721847360552633389b0765605b4bf78 (diff) |
wctdm24xxp: Dynamically allocate the board_name.
Move the data backing the mostly unused string away from the active
members of 'struct wctdm'.
The location where some of the other members of 'struct wctdm' are initialized
were moved so that wctdm_back_out_gracefully always has a fully formed
structure to work on.
Signed-off-by: Shaun Ruffell <sruffell@digium.com>
git-svn-id: http://svn.asterisk.org/svn/dahdi/linux/trunk@9955 a0bf4364-ded3-4de4-8d8a-66a801d63aff
Diffstat (limited to 'drivers')
-rw-r--r-- | drivers/dahdi/wctdm24xxp/base.c | 35 | ||||
-rw-r--r-- | drivers/dahdi/wctdm24xxp/wctdm24xxp.h | 2 |
2 files changed, 20 insertions, 17 deletions
diff --git a/drivers/dahdi/wctdm24xxp/base.c b/drivers/dahdi/wctdm24xxp/base.c index 1cc28c0..d092edb 100644 --- a/drivers/dahdi/wctdm24xxp/base.c +++ b/drivers/dahdi/wctdm24xxp/base.c @@ -4047,6 +4047,7 @@ static void wctdm_back_out_gracefully(struct wctdm *wc) kfree(frame); } + kfree(wc->board_name); kfree(wc); } @@ -4594,6 +4595,22 @@ __wctdm_init_one(struct pci_dev *pdev, const struct pci_device_id *ent) } up(&ifacelock); + wc->desc = (struct wctdm_desc *)ent->driver_data; + + /* This is to insure that the analog span is given lowest priority */ + sema_init(&wc->syncsem, 1); + INIT_LIST_HEAD(&wc->frame_list); + spin_lock_init(&wc->frame_list_lock); + init_waitqueue_head(&wc->regq); + spin_lock_init(&wc->reglock); + wc->oldsync = -1; + + wc->board_name = kasprintf(GFP_KERNEL, "%s%d", wctdm_driver.name, i); + if (!wc->board_name) { + wctdm_back_out_gracefully(wc); + return -ENOMEM; + } + #ifdef CONFIG_VOICEBUS_ECREFERENCE for (i = 0; i < ARRAY_SIZE(wc->ec_reference); ++i) { /* 256 is the smallest power of 2 that will contains the @@ -4609,17 +4626,6 @@ __wctdm_init_one(struct pci_dev *pdev, const struct pci_device_id *ent) } #endif - - wc->desc = (struct wctdm_desc *)ent->driver_data; - - /* This is to insure that the analog span is given lowest priority */ - wc->oldsync = -1; - sema_init(&wc->syncsem, 1); - INIT_LIST_HEAD(&wc->frame_list); - spin_lock_init(&wc->frame_list_lock); - - snprintf(wc->board_name, sizeof(wc->board_name)-1, "%s%d", wctdm_driver.name, i); - pci_set_drvdata(pdev, wc); wc->vb.ops = &voicebus_operations; wc->vb.pdev = pdev; @@ -4636,7 +4642,7 @@ __wctdm_init_one(struct pci_dev *pdev, const struct pci_device_id *ent) } if (ret) { - kfree(wc); + wctdm_back_out_gracefully(wc); return ret; } @@ -4644,9 +4650,6 @@ __wctdm_init_one(struct pci_dev *pdev, const struct pci_device_id *ent) voicebus_lock_latency(&wc->vb); - init_waitqueue_head(&wc->regq); - - spin_lock_init(&wc->reglock); wc->mods_per_board = NUM_MODULES; wc->pos = i; wc->txident = 1; @@ -4682,7 +4685,7 @@ __wctdm_init_one(struct pci_dev *pdev, const struct pci_device_id *ent) ret = hx8_check_firmware(wc); if (ret) { voicebus_release(&wc->vb); - kfree(wc); + wctdm_back_out_gracefully(wc); return -EIO; } diff --git a/drivers/dahdi/wctdm24xxp/wctdm24xxp.h b/drivers/dahdi/wctdm24xxp/wctdm24xxp.h index e3c5900..8b95ec5 100644 --- a/drivers/dahdi/wctdm24xxp/wctdm24xxp.h +++ b/drivers/dahdi/wctdm24xxp/wctdm24xxp.h @@ -153,7 +153,7 @@ struct wctdm_chan { struct wctdm { const struct wctdm_desc *desc; - char board_name[80]; + const char *board_name; int pos; /* card number in system */ spinlock_t frame_list_lock; |