From 58959676012bd1636cadc72415aa18c728287fc1 Mon Sep 17 00:00:00 2001 From: Volker Lendecke Date: Sat, 13 May 2006 17:17:07 +0000 Subject: r15569: Fix Coverity bug # 287. Jerry, can you check if WERR_NOMEM is a correct error code here? Thanks, Volker (This used to be commit 5787bd0ee90b081ae12a4a976893cb297fa7bed4) --- source3/printing/nt_printing.c | 11 +++++++++++ 1 file changed, 11 insertions(+) (limited to 'source3') diff --git a/source3/printing/nt_printing.c b/source3/printing/nt_printing.c index b0bac86dad..79061ebd41 100644 --- a/source3/printing/nt_printing.c +++ b/source3/printing/nt_printing.c @@ -3020,8 +3020,19 @@ static WERROR nt_printer_publish_ads(ADS_STRUCT *ads, /* build the ads mods */ ctx = talloc_init("nt_printer_publish_ads"); + if (ctx == NULL) { + SAFE_FREE(prt_dn); + return WERR_NOMEM; + } + mods = ads_init_mods(ctx); + if (mods == NULL) { + SAFE_FREE(prt_dn); + talloc_destroy(ctx); + return WERR_NOMEM; + } + get_local_printer_publishing_data(ctx, &mods, printer->info_2->data); ads_mod_str(ctx, &mods, SPOOL_REG_PRINTERNAME, printer->info_2->sharename); -- cgit