r22820 - gnucash/trunk/src/import-export/aqbanking - Minor bugfix: Guard against NULL pointer in sprintf argument.

[Christian Stimming]

Author: cstim
Date: 2013-03-02 08:49:06 -0500 (Sat, 02 Mar 2013)
New Revision: 22820
Trac: http://svn.gnucash.org/trac/changeset/22820

Modified:
   gnucash/trunk/src/import-export/aqbanking/dialog-ab-trans.c
Log:
Minor bugfix: Guard against NULL pointer in sprintf argument.

Modified: gnucash/trunk/src/import-export/aqbanking/dialog-ab-trans.c
===================================================================
--- gnucash/trunk/src/import-export/aqbanking/dialog-ab-trans.c	2013-03-02 00:25:28 UTC (rev 22819)
+++ gnucash/trunk/src/import-export/aqbanking/dialog-ab-trans.c	2013-03-02 13:49:06 UTC (rev 22820)
@@ -542,6 +542,8 @@
         if (!localBIC || !localIBAN
                 || (strlen(localBIC) == 0) || (strlen(localIBAN) == 0))
         {
+            const char* localBankCode = AB_Transaction_GetLocalBankCode(td->ab_trans);
+            const char* localAccountCode = AB_Transaction_GetLocalAccountNumber(td->ab_trans);
             values_ok = FALSE;
             gnc_error_dialog(td->dialog,
                              _("Your local bank account does not yet have the SEPA account information stored. "
@@ -549,8 +551,8 @@
                                "which has not yet been implemented directly in gnucash. "
                                "Please execute the command line program \"aqhbci-tool\" for your account, as follows: "
                                "aqhbci-tool4 getaccsepa -b %s -a %s"),
-                             AB_Transaction_GetLocalBankCode(td->ab_trans),
-                             AB_Transaction_GetLocalAccountNumber(td->ab_trans));
+                             (localBankCode ? localBankCode : ""),
+                             (localAccountCode ? localAccountCode : ""));
         }
     }